fix(eslint): do not set files if ignores is set (#2552)

https://eslint.org/docs/latest/use/configure/ignore#ignoring-files
AmadeusITGroup · Dec 5, 2024 · 352c82a · 352c82a
2 parents e6dd36e + 2f75fbc
commit 352c82a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -2,15 +2,15 @@ import {
   dirname,
   posix,
   relative,
-  sep
+  sep,
 } from 'node:path';
 import {
   fileURLToPath,
-  pathToFileURL
+  pathToFileURL,
 } from 'node:url';
 import shared from './eslint.shared.config.mjs';
 import {
-  sync
+  sync,
 } from 'globby';
 
 const __filename = fileURLToPath(import.meta.url);
@@ -43,11 +43,10 @@ const mergeESLintConfigs = async (globs) => {
     localConfigs = localConfigs.concat(
       configArray.map((config) => ({
         ...config,
-        files: (config.files || ['**/*']).flat().map((pathGlob) => addPrefix(directory, pathGlob)),
         ...(
           config.ignores
             ? { ignores: config.ignores.map((pathGlob) => addPrefix(directory, pathGlob)) }
-            : {}
+            : { files: (config.files || ['**/*']).flat().map((pathGlob) => addPrefix(directory, pathGlob)) }
         )
       }))
     );

diff --git a/...-config/schematics/ng-add/eslint/templates/workspace/eslint.config.__extension__.template b/...-config/schematics/ng-add/eslint/templates/workspace/eslint.config.__extension__.template
@@ -48,11 +48,10 @@ const mergeESLintConfigs = async (globs) => {
     localConfigs = localConfigs.concat(
       configArray.map((config) => ({
         ...config,
-        files: (config.files || ['**/*']).flat().map((pathGlob) => addPrefix(directory, pathGlob)),
         ...(
           config.ignores
             ? { ignores: config.ignores.map((pathGlob) => addPrefix(directory, pathGlob)) }
-            : {}
+            : { files: (config.files || ['**/*']).flat().map((pathGlob) => addPrefix(directory, pathGlob)) }
         )
       }))
     );