add TODO for rules in recommended ESLint 9 config

Once we upgrade to ESLint v9, the recommended config will enable several rules by default. There'll be no need to enable these explicitly. eslint/eslint@6ee3e9e Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid · Jun 17, 2024 · 1fa6f84 · 1fa6f84
1 parent ce6bda9
commit 1fa6f84
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/rules/best-practices.js b/rules/best-practices.js
@@ -12,7 +12,7 @@ module.exports = {
     "curly": ["error", "all"],
 
     // disallow empty static blocks
-    // TODO [eslint-config-airbnb-base@>=16]: Remove if enabled
+    // TODO [eslint@>=9]: Remove.  Included in default config.
     "no-empty-static-block": "error",
 
     // disallow unnecessary parentheses

diff --git a/rules/possible-errors.js b/rules/possible-errors.js
@@ -6,7 +6,7 @@
 module.exports = {
   "rules": {
     // disallow expressions where the operation doesn't affect the value
-    // TODO [eslint-config-airbnb-base@>=16]: Remove if enabled
+    // TODO [eslint@>=9]: Remove.  Included in default config.
     "no-constant-binary-expression": "error",
 
     // disallow constant expressions in conditions
@@ -23,6 +23,7 @@ module.exports = {
     "no-new-native-nonconstructor": "error",
 
     // disallow unused private class members
+    // TODO [eslint@>=9]: Remove.  Included in default config.
     "no-unused-private-class-members": "error"
   }
 };