@rushstack/eslint-config 3.7.1 → 4.0.1

package/CHANGELOG.json

@@ -1,6 +1,30 @@
 {
   "name": "@rushstack/eslint-config",
   "entries": [
+    {
+      "version": "4.0.1",
+      "tag": "@rushstack/eslint-config_v4.0.1",
+      "date": "Wed, 14 Aug 2024 22:37:32 GMT",
+      "comments": {
+        "dependency": [
+          {
+            "comment": "Updating dependency \"@rushstack/eslint-plugin\" to `0.16.0`"
+          }
+        ]
+      }
+    },
+    {
+      "version": "4.0.0",
+      "tag": "@rushstack/eslint-config_v4.0.0",
+      "date": "Tue, 13 Aug 2024 18:17:05 GMT",
+      "comments": {
+        "major": [
+          {
+            "comment": "[BREAKING CHANGE] Bump \"@typescript-eslint/eslint-plugin\" to \"~8.1.0\" and \"@typescript-eslint/eslint-parser\" to \"~8.1.0\". Due to these changes, node@>=17.0.0 and eslint@^8.57.0 are now required due to breaking changes in the newer rules set."
+          }
+        ]
+      }
+    },
     {
       "version": "3.7.1",
       "tag": "@rushstack/eslint-config_v3.7.1",

package/CHANGELOG.md

@@ -1,6 +1,18 @@
 # Change Log - @rushstack/eslint-config
 
-This log was last generated on Sat, 27 Jul 2024 00:10:27 GMT and should not be manually modified.
+This log was last generated on Wed, 14 Aug 2024 22:37:32 GMT and should not be manually modified.
+
+## 4.0.1
+Wed, 14 Aug 2024 22:37:32 GMT
+
+_Version update only_
+
+## 4.0.0
+Tue, 13 Aug 2024 18:17:05 GMT
+
+### Breaking changes
+
+- [BREAKING CHANGE] Bump "@typescript-eslint/eslint-plugin" to "~8.1.0" and "@typescript-eslint/eslint-parser" to "~8.1.0". Due to these changes, node@>=17.0.0 and eslint@^8.57.0 are now required due to breaking changes in the newer rules set.
 
 ## 3.7.1
 Sat, 27 Jul 2024 00:10:27 GMT

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rushstack/eslint-config",
-  "version": "3.7.1",
+  "version": "4.0.1",
   "description": "A TypeScript ESLint ruleset designed for large teams and projects",
   "license": "MIT",
   "repository": {
@@ -19,19 +19,19 @@
     "typescript"
   ],
   "peerDependencies": {
-    "eslint": "^6.0.0 || ^7.0.0 || ^8.0.0",
+    "eslint": "^8.57.0",
     "typescript": ">=4.7.0"
   },
   "dependencies": {
-    "@typescript-eslint/eslint-plugin": "~6.19.0",
-    "@typescript-eslint/utils": "~6.19.0",
-    "@typescript-eslint/parser": "~6.19.0",
-    "@typescript-eslint/typescript-estree": "~6.19.0",
+    "@typescript-eslint/eslint-plugin": "~8.1.0",
+    "@typescript-eslint/utils": "~8.1.0",
+    "@typescript-eslint/parser": "~8.1.0",
+    "@typescript-eslint/typescript-estree": "~8.1.0",
     "eslint-plugin-promise": "~6.1.1",
     "eslint-plugin-react": "~7.33.2",
     "eslint-plugin-tsdoc": "~0.3.0",
     "@rushstack/eslint-patch": "1.10.4",
-    "@rushstack/eslint-plugin": "0.15.2",
+    "@rushstack/eslint-plugin": "0.16.0",
     "@rushstack/eslint-plugin-packlets": "0.9.2",
     "@rushstack/eslint-plugin-security": "0.8.2"
   },

package/profile/_common.js

@@ -260,54 +260,10 @@ function buildRules(profile) {
           '@typescript-eslint/adjacent-overload-signatures': 'warn',
 
           // STANDARDIZED BY:   @typescript-eslint\eslint-plugin\dist\configs\recommended.json
-          //
-          // CONFIGURATION:     By default, these are banned: String, Boolean, Number, Object, Symbol
-          '@typescript-eslint/ban-types': [
-            'warn',
-            {
-              extendDefaults: false, // (the complete list is in this file)
-              types: {
-                String: {
-                  message: 'Use "string" instead',
-                  fixWith: 'string'
-                },
-                Boolean: {
-                  message: 'Use "boolean" instead',
-                  fixWith: 'boolean'
-                },
-                Number: {
-                  message: 'Use "number" instead',
-                  fixWith: 'number'
-                },
-                Object: {
-                  message: 'Use "object" instead, or else define a proper TypeScript type:'
-                },
-                Symbol: {
-                  message: 'Use "symbol" instead',
-                  fixWith: 'symbol'
-                },
-                Function: {
-                  message: [
-                    'The "Function" type accepts any function-like value.',
-                    'It provides no type safety when calling the function, which can be a common source of bugs.',
-                    'It also accepts things like class declarations, which will throw at runtime as they will not be called with "new".',
-                    'If you are expecting the function to accept certain arguments, you should explicitly define the function shape.'
-                  ].join('\n')
-                }
-
-                // This is a good idea, but before enabling it we need to put some thought into the recommended
-                // coding practices; the default suggestions are too vague.
-                //
-                // '{}': {
-                //   message: [
-                //     '"{}" actually means "any non-nullish value".',
-                //     '- If you want a type meaning "any object", you probably want "Record<string, unknown>" instead.',
-                //     '- If you want a type meaning "any value", you probably want "unknown" instead.'
-                //   ].join('\n')
-                // }
-              }
-            }
-          ],
+          '@typescript-eslint/no-unsafe-function-type': 'warn',
+
+          // STANDARDIZED BY:   @typescript-eslint\eslint-plugin\dist\configs\recommended.json
+          '@typescript-eslint/no-wrapper-object-types': 'warn',
 
           // RATIONALE:         We require "x as number" instead of "<number>x" to avoid conflicts with JSX.
           '@typescript-eslint/consistent-type-assertions': 'warn',
@@ -433,7 +389,9 @@ function buildRules(profile) {
               // Unused function arguments often indicate a mistake in JavaScript code.  However in TypeScript code,
               // the compiler catches most of those mistakes, and unused arguments are fairly common for type signatures
               // that are overriding a base class method or implementing an interface.
-              args: 'none'
+              args: 'none',
+              // Unused error arguments are common and useful for inspection when a debugger is attached.
+              caughtErrors: 'none'
             }
           ],