fix(misc): format must exclude ignored patterns (#18670)

Co-authored-by: Stefano Pongelli <stefano.pongelli@eligamo.com>
nrwl · Aug 22, 2023 · c479a54 · c479a54 · vercel · Aug 22, 2023
1 parent a25cc29
commit c479a54
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/packages/nx/src/command-line/format/format.ts b/packages/nx/src/command-line/format/format.ts
@@ -6,6 +6,7 @@ import {
   parseFiles,
   splitArgsIntoNxArgsAndOverrides,
 } from '../../utils/command-line-utils';
+import { getIgnoreObject } from '../../utils/ignore';
 import { fileExists, readJsonFile, writeJsonFile } from '../../utils/fileutils';
 import { calculateFileChanges, FileData } from '../../project-graph/file-utils';
 import * as yargs from 'yargs';
@@ -94,9 +95,16 @@ async function getPatterns(
       (f) => fileExists(f) && supportedExtensions.includes(path.extname(f))
     );
 
+    // exclude patterns in .nxignore or .gitignore
+    const nonIgnoredPatterns = getIgnoreObject().filter(patterns);
+
     return args.libsAndApps
-      ? await getPatternsFromApps(patterns, await allFileData(), graph)
-      : patterns;
+      ? await getPatternsFromApps(
+          nonIgnoredPatterns,
+          await allFileData(),
+          graph
+        )
+      : nonIgnoredPatterns;
   } catch {
     return allFilesPattern;
   }