Skip folders in CLI pattern (#3411)

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) {

tests_integration/__tests__/__snapshots__/skip-folders.js.snap

@@ -0,0 +1,17 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`skip folders passed specifically (stdout) 1`] = `
+"a/file.js
+b/file.js
+"
+`;
+
+exports[`skip folders passed specifically (write) 1`] = `Array []`;
+
+exports[`skips folders in glob (stdout) 1`] = `
+"a/file.js
+b/file.js
+"
+`;
+
+exports[`skips folders in glob (write) 1`] = `Array []`;

tests_integration/__tests__/skip-folders.js

@@ -0,0 +1,22 @@
+"use strict";
+
+const runPrettier = require("../runPrettier");
+
+expect.addSnapshotSerializer(require("../path-serializer"));
+
+describe("skips folders in glob", () => {
+  runPrettier("cli/skip-folders", ["**/*", "-l"]).test({
+    status: 1,
+    stderr: ""
+  });
+});
+
+describe("skip folders passed specifically", () => {
+  runPrettier("cli/skip-folders", [
+    "a",
+    "a/file.js",
+    "b",
+    "b/file.js",
+    "-l"
+  ]).test({ status: 1, stderr: "" });
+});

tests_integration/cli/skip-folders/a/file.js

@@ -0,0 +1 @@
+fooA(     )

tests_integration/cli/skip-folders/b/file.js

@@ -0,0 +1 @@
+fooB(   )