Merge pull request #33 from hildjj/ignore

Ignore rules tweaked
peggyjs · Aug 1, 2024 · a8f4010 · a8f4010
2 parents 7a9b0e9 + ec596fe
commit a8f4010
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/.npmignore b/.npmignore
@@ -1,5 +1,5 @@
 .github/
 .ncurc
-eslint.config.js
+eslint.config.mjs
 node_modules/
 test/
diff --git a/flat/modern.js b/flat/modern.js
@@ -5,6 +5,6 @@ const globals = require("globals");
 module.exports = {
   languageOptions: {
     globals: globals.node,
-    ecmaVersion: 2020,
+    ecmaVersion: 2022,
   },
 };
diff --git a/rules/js.js b/rules/js.js
@@ -96,7 +96,13 @@ module.exports = {
     "no-unsafe-negation": "error",
     "no-unsafe-optional-chaining": "error",
     "no-unused-private-class-members": "off", // Not using private yet
-    "no-unused-vars": ["error", { "varsIgnorePattern": "^_" }],
+    "no-unused-vars": ["error", {
+      argsIgnorePattern: "^_",
+      caughtErrorsIgnorePattern: "^(_|ignore)",
+      destructuredArrayIgnorePattern: "^_",
+      varsIgnorePattern: "^_[^_]",
+      reportUsedIgnorePattern: true,
+    }],
 
     // This is going to cause problems, but I don't care.
     "no-use-before-define": "error",