@prowise/eslint-config-linting-rules 0.0.9 → 0.0.10

package/package.json

@@ -1 +1 @@
-{"name":"@prowise/eslint-config-linting-rules","version":"0.0.9","description":"Default Prowise linting rules for npm packages.","main":"index.js","exports":{"./typescript":"./typescript.js","./backend":"./backend.js","./frontend":"./frontend.js"},"scripts":{"lint":"eslint **/*.js"},"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":{"@typescript-eslint/eslint-plugin":"^5.36.2","@typescript-eslint/parser":"^5.36.2","confusing-browser-globals":"^1.0.11","eslint":"^8.24.0","eslint-import-resolver-typescript":"^3.5.1","eslint-plugin-import":"^2.26.0","eslint-plugin-jest":"^27.0.1","eslint-plugin-jsdoc":"^39.3.6","eslint-plugin-node":"^11.1.0","eslint-plugin-prefer-arrow":"^1.2.3","eslint-plugin-prettier":"^4.2.1","eslint-plugin-react":"^7.31.7","eslint-plugin-react-hooks":"^4.6.0","object.assign":"^4.1.4","prettier":"^2.7.1"}}
+{"name":"@prowise/eslint-config-linting-rules","version":"0.0.10","description":"Default Prowise linting rules for npm packages.","main":"index.js","exports":{"./typescript":"./typescript.js","./backend":"./backend.js","./frontend":"./frontend.js"},"scripts":{"lint":"eslint **/*.js"},"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":{"@typescript-eslint/eslint-plugin":"^5.36.2","@typescript-eslint/parser":"^5.36.2","confusing-browser-globals":"^1.0.11","eslint":"^8.24.0","eslint-import-resolver-typescript":"^3.5.1","eslint-plugin-import":"^2.26.0","eslint-plugin-jest":"^27.0.1","eslint-plugin-jsdoc":"^39.3.6","eslint-plugin-node":"^11.1.0","eslint-plugin-prefer-arrow":"^1.2.3","eslint-plugin-prettier":"^4.2.1","eslint-plugin-react":"^7.31.7","eslint-plugin-react-hooks":"^4.6.0","object.assign":"^4.1.4","prettier":"^2.7.1"}}

package/src/rules/base-rules.js

@@ -364,8 +364,9 @@ module.exports = {
         /**
          * Enforces no braces where they can be omitted 🔧
          * https://eslint.org/docs/latest/rules/arrow-body-style.
+         * Turned off => conflicting with the prettier rules.
          */
-        "arrow-body-style": ["error", "as-needed", { requireReturnForObjectLiteral: false }],
+        "arrow-body-style": "off",
 
         /**
          * Treat var statements as if they were block scoped
@@ -624,8 +625,9 @@ module.exports = {
         /**
          * Disallow arrow functions where they could be confused with comparisons 🔧
          * https://eslint.org/docs/latest/rules/no-confusing-arrow.
+         * Turned off => conflicting with the prettier rules.
          */
-        "no-confusing-arrow": ["error", { allowParens: true }],
+        "no-confusing-arrow": "off",
 
         /**
          * Disallow use of console
@@ -861,8 +863,9 @@ module.exports = {
         /**
          * Disallow nested ternary expressions
          * https://eslint.org/docs/latest/rules/no-nested-ternary.
+         * Turned off => conflicting with the prettier rules.
          */
-        "no-nested-ternary": "error",
+        "no-nested-ternary": "off",
 
         /**
          * Disallow use of new operator when not part of the assignment or comparison
@@ -908,29 +911,9 @@ module.exports = {
 
         /**
          * Disallow reassignment of function parameters
-         * /**
-         * disallow parameter object manipulation except for specific exclusions
          * https://eslint.org/docs/latest/rules/no-param-reassign.
          */
-        "no-param-reassign": [
-            "error",
-            {
-                props: true,
-                ignorePropertyModificationsFor: [
-                    "acc", // for reduce accumulators
-                    "accumulator", // for reduce accumulators
-                    "e", // for e.returnvalue
-                    "ctx", // for Koa routing
-                    "context", // for Koa routing
-                    "req", // for Express requests
-                    "request", // for Express requests
-                    "res", // for Express responses
-                    "response", // for Express responses
-                    "$scope", // for Angular 1 scopes
-                    "staticContext", // for ReactRouter context
-                ],
-            },
-        ],
+        "no-param-reassign": ["error", { props: false }],
 
         /**
          * Disallow use of unary operators, ++ and --
@@ -1273,8 +1256,9 @@ module.exports = {
         /**
          * Suggest using arrow functions as callbacks 🔧
          * https://eslint.org/docs/latest/rules/prefer-arrow-callback.
+         * Turned off => conflicting with the prettier rules.
          */
-        "prefer-arrow-callback": ["error", { allowNamedFunctions: false, allowUnboundThis: true }],
+        "prefer-arrow-callback": "off",
 
         /**
          * Suggest using of const declaration for variables that are never modified after declared 🔧

package/src/rules/plugin-react-rules.js

@@ -301,7 +301,7 @@ module.exports = {
          * Disallow creating unstable components inside components
          * https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-unstable-nested-components.md.
          */
-        "react/no-unstable-nested-components": "error",
+        "react/no-unstable-nested-components": ["error", { allowProps: true }],
 
         /**
          * Disallow declaring unused methods of component class
@@ -634,7 +634,7 @@ module.exports = {
          * Disallow problematic leaked values from being rendered 🔧
          * https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-no-leaked-render.md.
          */
-        "react/jsx-no-leaked-render": "warn",
+        "react/jsx-no-leaked-render": ["warn", { validStrategies: ["coerce", "ternary"] }],
 
         /**
          * Disallow usage of string literals in JSX

package/src/rules/plugin-typescript-rules.js

@@ -349,7 +349,7 @@ module.exports = {
          * Disallow classes used as namespaces 🔒
          * https://typescript-eslint.io/rules/no-extraneous-class.
          */
-        "@typescript-eslint/no-extraneous-class": "warn",
+        "@typescript-eslint/no-extraneous-class": "off",
 
         /**
          * Require Promise-like statements to be handled appropriately ✅ 💡 💭
@@ -428,7 +428,7 @@ module.exports = {
          * Disallow Promises in places not designed to handle them ✅ 💭
          * https://typescript-eslint.io/rules/no-misused-promises.
          */
-        "@typescript-eslint/no-misused-promises": "error",
+        "@typescript-eslint/no-misused-promises": ["error", { checksVoidReturn: false }],
 
         /**
          * Disallow custom TypeScript modules and namespaces ✅