chore: enable unused-vars caughtErrors: all internally (#8369)

typescript-eslint · Feb 22, 2024 · 813ad93 · 813ad93
1 parent b17c453
commit 813ad93
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -158,7 +158,11 @@ export default tseslint.config(
       ],
       '@typescript-eslint/no-unused-vars': [
         'error',
-        { varsIgnorePattern: '^_', argsIgnorePattern: '^_' },
+        {
+          caughtErrors: 'all',
+          varsIgnorePattern: '^_',
+          argsIgnorePattern: '^_',
+        },
       ],
       '@typescript-eslint/prefer-nullish-coalescing': [
         'error',
@@ -472,7 +476,7 @@ export default tseslint.config(
     files: ['packages/ast-spec/src/**/*.{ts,tsx,cts,mts}'],
     rules: {
       // disallow ALL unused vars
-      '@typescript-eslint/no-unused-vars': 'error',
+      '@typescript-eslint/no-unused-vars': ['error', { caughtErrors: 'all' }],
       '@typescript-eslint/sort-type-constituents': 'error',
     },
   },

diff --git a/packages/website/src/hooks/useMediaQuery.ts b/packages/website/src/hooks/useMediaQuery.ts
@@ -22,7 +22,7 @@ const useMediaQuery = (mediaQuery: string): boolean => {
 
     try {
       mediaQueryList.addEventListener('change', documentChangeHandler);
-    } catch (e) {
+    } catch {
       // Safari isn't supporting mediaQueryList.addEventListener
       // eslint-disable-next-line deprecation/deprecation
       mediaQueryList.addListener(documentChangeHandler);
@@ -32,7 +32,7 @@ const useMediaQuery = (mediaQuery: string): boolean => {
     return () => {
       try {
         mediaQueryList.removeEventListener('change', documentChangeHandler);
-      } catch (e) {
+      } catch {
         // Safari isn't supporting mediaQueryList.removeEventListener
         // eslint-disable-next-line deprecation/deprecation
         mediaQueryList.removeListener(documentChangeHandler);