@josundt/eslint-config 5.2.2 → 5.2.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@josundt/eslint-config",
-  "version": "5.2.2",
+  "version": "5.2.10",
   "description": "ESLint ruleset with required plugins for josundt TypeScript projects",
   "main": "index.js",
   "scripts": {
@@ -27,22 +27,22 @@
     "utils/**/*.js"
   ],
   "peerDependencies": {
-    "typescript": ">=5.0.4"
+    "typescript": ">=5.2.2"
   },
   "dependencies": {
-    "@josundt/prettier-config": "^1.0.0",
-    "@typescript-eslint/eslint-plugin": "5.59.5",
-    "@typescript-eslint/parser": "5.59.5",
-    "eslint": "8.40.0",
-    "eslint-import-resolver-typescript": "3.5.5",
-    "eslint-plugin-deprecation": "1.4.1",
+    "@josundt/prettier-config": "^3.0.3",
+    "@typescript-eslint/eslint-plugin": "6.5.0",
+    "@typescript-eslint/parser": "6.5.0",
+    "eslint": "8.48.0",
+    "eslint-import-resolver-typescript": "3.6.0",
+    "eslint-plugin-deprecation": "1.5.0",
     "eslint-plugin-eslint-comments": "3.2.0",
-    "eslint-plugin-import": "2.27.5",
+    "eslint-plugin-import": "2.28.1",
     "eslint-plugin-jasmine": "4.1.3",
-    "eslint-plugin-jest": "27.2.1",
-    "eslint-plugin-jsdoc": "44.0.2",
-    "eslint-plugin-no-lookahead-lookbehind-regexp": "0.1.0",
-    "eslint-plugin-prettier": "4.2.1",
-    "eslint-plugin-unicorn": "47.0.0"
+    "eslint-plugin-jest": "27.2.3",
+    "eslint-plugin-jsdoc": "46.5.0",
+    "eslint-plugin-no-lookahead-lookbehind-regexp": "0.3.0",
+    "eslint-plugin-prettier": "5.0.0",
+    "eslint-plugin-unicorn": "48.0.1"
   }
 }

package/rules/eslint.js

@@ -3,7 +3,7 @@ module.exports = {
     rules: {
         "arrow-body-style": "error",
         "arrow-parens": ["error", "as-needed"],
-        "class-methods-use-this": "off", // Warn when methods could be static
+        "class-methods-use-this": ["off", { enforceForClassFields: true }], // Warn when methods could be static
         "complexity": ["warn", { max: 20 }],
         "curly": "error",
         "default-case-last": "error",

package/rules/typescript-eslint/extensionrules.js

@@ -5,8 +5,8 @@ const eslintRules = eslintRuleSet.rules;
 
 // Map of all the typescript-eslint extensions.
 // If extension rule has additional properties compared to standard eslint rule:
-// If the map value is null:
-//     The options from the standard rule will be used as is.
+// If the map value is a boolean:
+//     If true the options from the standard rule will be used as is; if false the rule will be switched off.
 // If the map value is an object:
 //     The object will be merged merged with standard rule (rule object 1).
 // If the map value is an array:
@@ -15,6 +15,14 @@ const eslintRules = eslintRuleSet.rules;
 //     The standard eslint options object will be passed as parameters, the return statement will be added as options.
 
 const extensions = new Map([
+    [
+        "class-methods-use-this",
+        false
+        // {
+        //     ignoreOverrideMethods: false,
+        //     ignoreClassesThatImplementAnInterface: false
+        // }
+    ],
     ["default-param-last", true],
     [
         "dot-notation",
@@ -33,7 +41,7 @@ const extensions = new Map([
     ],
     ["no-array-constructor", true],
     ["no-dupe-class-members", true],
-    ["no-duplicate-imports", true],
+    ["no-duplicate-imports", false], // typescript-eslint 5.25.0: deprecated in favour of import/no-duplicates
     [
         "no-empty-function",
         {

package/rules/typescript-eslint/rules.js

@@ -21,7 +21,12 @@ const rules = {
         }
     ],
     "ban-tslint-comment": "error", // No longer use tslint - remove rules
-    "ban-types": "off", // Can be used to ban certain types
+    "ban-types": [
+        "error",
+        {
+            // Enable additional/disable default disabled types here
+        }
+    ],
     "class-literal-property-style": "off",
     "consistent-generic-constructors": ["off", "constructor"],
     "consistent-indexed-object-style": ["error", "record"],
@@ -191,7 +196,6 @@ const rules = {
             allowThrowingUnknown: true // Default is to allow throwing values of type unknown
         }
     ],
-    "no-type-alias": "off",
     "no-unnecessary-boolean-literal-compare": "error",
     "no-unnecessary-condition": "off", // allow runtime null checks etc even if reported not necessary by type system
     "no-unnecessary-qualifier": "error",
@@ -226,7 +230,13 @@ const rules = {
         {
             ignoreConditionalTests: true,
             ignoreTernaryTests: true,
-            ignoreMixedLogicalExpressions: true
+            ignoreMixedLogicalExpressions: true,
+            ignorePrimitives: {
+                string: false,
+                number: false,
+                bigint: false,
+                boolean: false
+            }
         }
     ],
     "prefer-optional-chain": "error",
@@ -238,11 +248,16 @@ const rules = {
     "prefer-string-starts-ends-with": "error",
     "prefer-ts-expect-error": "error",
     "promise-function-async": "off",
-    "require-array-sort-compare": "error",
+    "require-array-sort-compare": [
+        "error",
+        {
+            ignoreStringArrays: false
+        }
+    ],
     "restrict-plus-operands": [
         "error",
         {
-            checkCompoundAssignments: true,
+            skipCompoundAssignments: false,
             allowAny: false
         }
     ],