feat: move to ESLint v9 👏

BREAKING CHANGE: Projects must now use ESLint v9 in order to use these rulesets.
strvcom · Jan 24, 2025 · 4de1c1f · 4de1c1f
1 parent 836cdc8
commit 4de1c1f
Show file tree

Hide file tree

Showing 12 changed files with 6,165 additions and 4,939 deletions.
diff --git a/eslint.config.js b/eslint.config.js
@@ -15,8 +15,6 @@ const configs = [{
 node,
 optional,
 style,
-{ files: ['**/*.js', '**/*.mjs'], languageOptions: { sourceType: 'module' } },
-{ files: ['**/*.cjs'], languageOptions: { sourceType: 'commonjs' } },
 { rules: {
   // TODO: Figure out why ESLint cannot resolve the @typescript-eslint packages
   'import/no-unresolved': 'off',

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -11,8 +11,8 @@
     "Korneliusz Dubieniecki <[email protected]>"
   ],
   "devDependencies": {
-    "@commitlint/cli": "^18.6.0",
-    "eslint": "^8.57.0",
+    "@commitlint/cli": "^19",
+    "eslint": "^9",
     "lerna": "^8.1.2",
     "remark-cli": "^12.0.0",
     "remark-preset-lint-consistent": "^5",

diff --git a/packages/eslint-config-base/index.mjs b/packages/eslint-config-base/index.mjs
@@ -590,9 +590,7 @@ const config = {
     'vars-on-top': 'warn',
 
     // Ensure an imported module can be resolved to a module on the local filesystem
-    'import/no-unresolved': ['error', {
-      commonjs: true,
-    }],
+    'import/no-unresolved': ['error'],
 
     // Verifies that all named imports are part of the set of named exports in the referenced module
     'import/named': 'error',
@@ -605,8 +603,7 @@ const config = {
     'import/first': 'error',
 
     // Enforces names exist at the time they are dereferenced, when imported as a full namespace
-    // TODO: Re-enable when eslint-plugin-import gets ESLint flat config support
-    // 'import/namespace': 'error',
+    'import/namespace': 'error',
 
     // Forbid import of modules using absolute paths
     // Node.js allows the import of modules using an absolute path such as */home/xyz/file.js*. That

diff --git a/packages/eslint-config-base/package.json b/packages/eslint-config-base/package.json
@@ -11,8 +11,8 @@
     "Danny Kijkov <[email protected]>"
   ],
   "dependencies": {
-    "@stylistic/eslint-plugin": "^1.6.0",
-    "eslint-plugin-import": "^2.27.0"
+    "@stylistic/eslint-plugin": "^2.13.0",
+    "eslint-plugin-import": "^2.31.0"
   },
   "engines": {
     "node": ">=20"
@@ -30,7 +30,7 @@
     "./style": "./style.mjs"
   },
   "peerDependencies": {
-    "eslint": "^8"
+    "eslint": "^9"
   },
   "publishConfig": {
     "access": "public"

diff --git a/packages/eslint-config-base/style.mjs b/packages/eslint-config-base/style.mjs
@@ -227,7 +227,7 @@ const config = {
 
     // Enforce a particular style for multiline comments
     // Prefer consecutive line comments for multiline comment block
-    'multiline-comment-style': ['warn', 'separate-lines'],
+    '@stylistic/multiline-comment-style': ['warn', 'separate-lines'],
 
     // Enforce newlines between operands of ternary expressions
     // Enforce either the whole ternary on a single line or each part of the ternary on its own line

diff --git a/packages/eslint-config-graphql/package.json b/packages/eslint-config-graphql/package.json
@@ -25,7 +25,7 @@
   ],
   "license": "BSD-3-Clause",
   "peerDependencies": {
-    "eslint": "^8"
+    "eslint": "^9"
   },
   "publishConfig": {
     "access": "public"

diff --git a/packages/eslint-config-mocha/package.json b/packages/eslint-config-mocha/package.json
@@ -12,7 +12,7 @@
   ],
   "dependencies": {
     "@strv/eslint-config-base": "^4.0.0",
-    "eslint-plugin-import": "^2.27.0",
+    "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-mocha": "^10.1.0"
   },
   "engines": {
@@ -30,7 +30,7 @@
   ],
   "license": "BSD-3-Clause",
   "peerDependencies": {
-    "eslint": "^8"
+    "eslint": "^9"
   },
   "publishConfig": {
     "access": "public"

diff --git a/packages/eslint-config-node/package.json b/packages/eslint-config-node/package.json
@@ -12,7 +12,7 @@
   ],
   "dependencies": {
     "@strv/eslint-config-base": "^4.0.0",
-    "eslint-plugin-n": "^16.6.2"
+    "eslint-plugin-n": "^17.15.0"
   },
   "engines": {
     "node": ">=20"
@@ -32,7 +32,7 @@
     "./style": "./style.mjs"
   },
   "peerDependencies": {
-    "eslint": "^8"
+    "eslint": "^9"
   },
   "publishConfig": {
     "access": "public"

diff --git a/packages/eslint-config-node/v20.mjs b/packages/eslint-config-node/v20.mjs
diff --git a/packages/eslint-config-react/package.json b/packages/eslint-config-react/package.json
@@ -16,7 +16,7 @@
     "@strv/eslint-config-base": "^4.0.0",
     "eslint-plugin-jsx-a11y": "^6.4.1",
     "eslint-plugin-react": "^7.23.0",
-    "eslint-plugin-react-hooks": "^4.2.0"
+    "eslint-plugin-react-hooks": "^5.0.0"
   },
   "engines": {
     "node": ">=20"
@@ -35,7 +35,7 @@
   ],
   "license": "BSD-3-Clause",
   "peerDependencies": {
-    "eslint": "^8"
+    "eslint": "^9"
   },
   "publishConfig": {
     "access": "public"

diff --git a/packages/eslint-config-typescript/package.json b/packages/eslint-config-typescript/package.json
@@ -12,8 +12,8 @@
   ],
   "dependencies": {
     "@strv/eslint-config-base": "^4.0.0",
-    "@typescript-eslint/eslint-plugin": "^7.0.0",
-    "@typescript-eslint/parser": "^7.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.0.0",
+    "@typescript-eslint/parser": "^8.0.0",
     "eslint-import-resolver-typescript": "^3.6.1"
   },
   "engines": {
@@ -34,7 +34,7 @@
   ],
   "license": "BSD-3-Clause",
   "peerDependencies": {
-    "eslint": "^8"
+    "eslint": "^9"
   },
   "publishConfig": {
     "access": "public"