Skip folders in CLI pattern

duailibe · duailibe · commit 639c5238859e · 2017-12-05T16:07:27.000-02:00
diff --git a/src/cli-util.js b/src/cli-util.js
@@ -281,7 +281,7 @@ function eachFilename(argv, patterns, callback) {
 
   try {
     const filePaths = globby
-      .sync(patterns, { dot: true })
+      .sync(patterns, { dot: true, nodir: true })
       .map(filePath => path.relative(process.cwd(), filePath));
 
     if (filePaths.length === 0) {
diff --git a/tests_integration/__tests__/__snapshots__/skip-folders.js.snap b/tests_integration/__tests__/__snapshots__/skip-folders.js.snap
@@ -0,0 +1,9 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`skips folders (stdout) 1`] = `
+"skip-folders/a/file.js
+skip-folders/b/file.js
+"
+`;
+
+exports[`skips folders (write) 1`] = `Array []`;
diff --git a/tests_integration/__tests__/skip-folders.js b/tests_integration/__tests__/skip-folders.js
@@ -0,0 +1,12 @@
+"use strict";
+
+const runPrettier = require("../runPrettier");
+
+expect.addSnapshotSerializer(require("../path-serializer"));
+
+describe("skips folders", () => {
+  runPrettier("cli", ["skip-folders/**/*", "-l"]).test({
+    status: 1,
+    stderr: ""
+  });
+});
diff --git a/tests_integration/cli/skip-folders/a/file.js b/tests_integration/cli/skip-folders/a/file.js
@@ -0,0 +1 @@
+fooA(     )
diff --git a/tests_integration/cli/skip-folders/b/file.js b/tests_integration/cli/skip-folders/b/file.js
@@ -0,0 +1 @@
+fooB(   )
