@wistia/eslint-config 2.5.1 → 2.5.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @wistia/eslint-config
 
+## 2.5.2
+
+### Patch Changes
+
+- [#473](https://github.com/wistia/eslint-config/pull/473) [`3cc15e5`](https://github.com/wistia/eslint-config/commit/3cc15e5b1baa827788023feac6c96dc9e08a5a86) Thanks [@okize](https://github.com/okize)! - fix: turn off `@typescript-eslint/no-unused-vars` as `noUnusedLocals` in tsconfig is preferred
+
 ## 2.5.1
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/eslint-config",
-  "version": "2.5.1",
+  "version": "2.5.2",
   "description": "Wistia's ESLint configurations",
   "packageManager": "yarn@4.13.0",
   "type": "module",

package/src/rules/typescript.mjs

@@ -313,7 +313,7 @@ export default {
 
   // Disallow this keywords outside of classes or class-like objects
   // https://typescript-eslint.io/rules/no-invalid-this
-  // decision: superceded by `noImplicitThis` in tsconfig
+  // decision: preference is to set `"noImplicitThis": true` in tsconfig
   '@typescript-eslint/no-invalid-this': 'off',
 
   // Disallow void type outside of generic or return types
@@ -488,18 +488,8 @@ export default {
   // Disallow unused variables
   // https://typescript-eslint.io/rules/no-unused-vars
   // decision: emulate the TypeScript style of exempting names starting with _
-  '@typescript-eslint/no-unused-vars': [
-    'error',
-    {
-      args: 'all',
-      argsIgnorePattern: '^_',
-      caughtErrors: 'all',
-      caughtErrorsIgnorePattern: '^_',
-      destructuredArrayIgnorePattern: '^_',
-      varsIgnorePattern: '^_',
-      ignoreRestSiblings: true,
-    },
-  ],
+  // decision: preference is to set `"noUnusedLocals": true` in tsconfig
+  '@typescript-eslint/no-unused-vars': 'off',
 
   // Disallow the use of variables before they are defined
   // https://typescript-eslint.io/rules/no-use-before-define

package/test/__snapshots__/typescript.mjs.snap

@@ -977,16 +977,7 @@
       2,
     ],
     "@typescript-eslint/no-unused-vars": [
-      2,
-      {
-        "args": "all",
-        "argsIgnorePattern": "^_",
-        "caughtErrors": "all",
-        "caughtErrorsIgnorePattern": "^_",
-        "destructuredArrayIgnorePattern": "^_",
-        "ignoreRestSiblings": true,
-        "varsIgnorePattern": "^_",
-      },
+      0,
     ],
     "@typescript-eslint/no-use-before-define": [
       2,