fix performance issue causing ignore-rules to be ignored

k-yle · k-yle · commit 0f78d764a160 · 2025-04-03T20:00:56.000+11:00
re eslint/eslint#19297
diff --git a/README.md b/README.md
@@ -66,5 +66,9 @@ Add the following to `package.json`:
   "editor.codeActionsOnSave": {
     "source.fixAll.eslint": "explicit",
   },
+
+  // optional, if the local vs global versions conflict.
+  // the exact syntax would depend on your package manager.
+  "prettier.prettierPath": "./node_modules/prettier",
 }
 ```
diff --git a/index.js b/index.js
@@ -24,8 +24,6 @@ import { selfClosingComp } from './rules/custom/self-closing-comp.js';
 
 /** @type {import('typescript-eslint').ConfigWithExtends[]} */
 const jsConfigs = [
-  includeIgnoreFile(join(process.cwd(), '.gitignore')),
-  { ignores: ['**/*.snap', '**/.next'], name: 'eslint-config-kyle/ignore' },
   eslint.configs.recommended,
   ...tsEslint.configs.strict,
   ...tsEslint.configs.stylistic,
@@ -76,6 +74,12 @@ const jsConfigs = [
 ];
 
 export default tsEslint.config(
+  // ignore everything that is not version-controlled
+  includeIgnoreFile(join(process.cwd(), '.gitignore')),
+
+  // and ignore some files that _are_ version-controlled
+  { ignores: ['**/*.snap'], name: 'eslint-config-kyle/ignore' },
+
   // we need to update all JS configs to only validate JS/TS files,
   // because other languages don't necessarily have a compatible
   // AST API...
