@prowise/eslint-config-linting-rules 2.4.6 → 3.0.0

package/README.md

@@ -42,9 +42,9 @@ export default [
 
 ```
 
-Sample of a `eslint.config.js` file for a frontend project using react and typescript:
+Sample of a `eslint.config.js` file for a react frontend project using react and typescript:
 ```
-import backend from "@prowise/eslint-config-linting-rules/frontend";
+import frontend from "@prowise/eslint-config-linting-rules/front-react";
 
 export default [
     { ignores: [ "**/reports", "**/dist/"] },
@@ -75,7 +75,7 @@ You need to remove all `.eslintrc` and `.eslintignore` files from all sub folder
 
 If you want different configuration for the different folders yo can use the following config as example:
 ```
-import frontend from "@prowise/eslint-config-linting-rules/frontend";
+import frontend from "@prowise/eslint-config-linting-rules/front-react";
 
 export default [
     // IGNORES MUST BE ON TOP !

package/base.js

@@ -1,8 +1,8 @@
 /*
 * Basic rules for all javascript projects
 */
-import progress from "eslint-plugin-file-progress";
 import base from "./rules/baseSettings.js";
+import progress from "./rules/file-progress.js";
 import imports from "./rules/plugin-import-rules.js";
 import jsDoc from "./rules/plugin-jsdoc-rules.js";
 import preferArrow from "./rules/plugin-prefer-arrow-rules.js";

package/eslint.config.js

@@ -1,9 +1,27 @@
 import base from "./base.js";
+import jsx from "./rules/jsx-pascal-case-rule.js";
+import jsxA11y from "./rules/plugin-jsx-a11y-rules.js";
+import next from "./rules/plugin-next-rules.js";
 import node from "./rules/plugin-node-rules.js";
+import react from "./rules/plugin-react-rules.js";
+import typescript from "./rules/plugin-typescript-rules.js";
+import vitest from "./rules/plugin-vitest-rules.js";
 
-export default [base, node, {
-    rules: {
-        "import/no-named-as-default-member": "off",
-        "import/no-named-as-default": "off",
-    },
-}];
+// this is only for testing all rules are configured correctly.
+// DO NOT USE this in any repository as some of the rules will give conflicting results
+export default [
+    base,
+    node,
+    typescript,
+    vitest,
+    react,
+    next,
+    jsxA11y,
+    {
+        plugins: { jsx },
+        rules: {
+            "import/no-named-as-default-member": "off",
+            "import/no-named-as-default": "off",
+            "jsx/jsx-element-pascal-case": "error",
+        },
+    }];

package/front-react.js

@@ -11,7 +11,7 @@ export default [
     react,
     vitest,
     {
-        plugins: { "@typescript-eslint": typescript.plugins["@typescript-eslint"], jsx },
+        plugins: { typescript: typescript.plugins.typescript, jsx },
         settings: { react: { version: "detect" } },
         languageOptions: {
             globals: {
@@ -22,7 +22,7 @@ export default [
         },
         rules: {
             "style/quote-props": ["error", "as-needed", { keywords: false, unnecessary: true, numbers: false }],
-            "@typescript-eslint/naming-convention": [
+            "typescript/naming-convention": [
                 "error",
                 { selector: "variable", format: ["camelCase", "PascalCase", "UPPER_CASE"] },
                 { selector: "typeLike", format: ["PascalCase"] },

package/front-solid.js

@@ -1,13 +1,11 @@
 import globals from "globals";
 import base from "./base.js";
-import solid from "./rules/plugin-solid-rules.js";
 import typescript from "./rules/plugin-typescript-rules.js";
 import vitest from "./rules/plugin-vitest-rules.js";
 
 export default [
     base,
     typescript,
-    solid,
     vitest,
     {
         languageOptions: {

package/next.js

@@ -13,7 +13,7 @@ export default [
     next,
     vitest,
     {
-        plugins: { "@typescript-eslint": typescript.plugins["@typescript-eslint"], jsx },
+        plugins: { typescript: typescript.plugins.typescript, jsx },
         settings: { react: { version: "detect" } },
         languageOptions: {
             globals: {
@@ -25,7 +25,7 @@ export default [
         },
         rules: {
             "style/quote-props": ["error", "as-needed", { keywords: false, unnecessary: true, numbers: false }],
-            "@typescript-eslint/naming-convention": [
+            "typescript/naming-convention": [
                 "error",
                 { selector: "variable", format: ["camelCase", "PascalCase", "UPPER_CASE"] },
                 { selector: "typeLike", format: ["PascalCase"] },

package/package.json

@@ -1 +1 @@
-{"name":"@prowise/eslint-config-linting-rules","version":"2.4.6","description":"Default Prowise linting rules for npm packages.","main":"index.js","type":"module","exports":{"./typescript":"./typescript.js","./backend":"./backend.js","./front-react":"./front-react.js","./front-solid":"./front-solid.js","./next":"./next.js"},"scripts":{"lint":"eslint","test":"node tests/jsx-pascal-case-rule.test.js","inspect":"eslint --inspect-config"},"repository":{"type":"git","url":"git+ssh://git@bitbucket.org/prowise/npm-linting-rules.git"},"author":"Prowise B.V.","license":"UNLICENSED","homepage":"https://bitbucket.org/prowise/npm-linting-rules#readme","dependencies":{"@next/eslint-plugin-next":"^16.2.1","@stylistic/eslint-plugin":"^5.10.0","@vitest/eslint-plugin":"^1.6.13","confusing-browser-globals":"^1.0.11","eslint":"^9.39.4","eslint-import-resolver-typescript":"^4.4.4","eslint-plugin-file-progress":"^3.0.2","eslint-plugin-import-x":"^4.16.2","eslint-plugin-jsdoc":"^62.8.1","eslint-plugin-jsx-a11y":"^6.10.2","eslint-plugin-n":"^17.24.0","eslint-plugin-prefer-arrow-functions":"^3.9.1","eslint-plugin-react":"^7.37.5","eslint-plugin-react-hooks":"^7.0.1","eslint-plugin-solid":"^0.14.5","globals":"^17.4.0","typescript-eslint":"^8.58.0"},"devDependencies":{"@eslint/config-inspector":"^1.5.0"}}
+{"name":"@prowise/eslint-config-linting-rules","version":"3.0.0","description":"Default Prowise linting rules for npm packages.","main":"index.js","type":"module","exports":{"./typescript":"./typescript.js","./backend":"./backend.js","./front-react":"./front-react.js","./front-solid":"./front-solid.js","./next":"./next.js"},"scripts":{"lint":"eslint","test":"node tests/*.test.js","inspect":"eslint --inspect-config"},"repository":{"type":"git","url":"git+ssh://git@bitbucket.org/prowise/npm-linting-rules.git"},"author":"Prowise B.V.","license":"UNLICENSED","homepage":"https://bitbucket.org/prowise/npm-linting-rules#readme","dependencies":{"@eslint-react/eslint-plugin":"^5.10.0","@htmlacademy/eslint-plugin-jsx-a11y":"^6.11.0","@next/eslint-plugin-next":"^16.2.10","@stylistic/eslint-plugin":"^5.10.0","@vitest/eslint-plugin":"^1.6.20","confusing-browser-globals":"^1.0.11","eslint":"^10.6.0","eslint-import-resolver-typescript":"^4.4.5","eslint-plugin-import-x":"^4.17.1","eslint-plugin-jsdoc":"^63.0.10","eslint-plugin-n":"^18.2.1","eslint-plugin-prefer-arrow-functions":"^3.9.1","globals":"^17.7.0","nanospinner":"^1.2.2","typescript-eslint":"^8.62.1"},"devDependencies":{"@eslint/config-inspector":"^3.0.4"}}