fix(prettierignore): support absolute filename (#2783)

src/cli-util.js

@@ -292,7 +292,14 @@ function eachFilename(argv, patterns, callback) {
   }
 
   try {
-    const filePaths = globby.sync(patterns, { dot: true });
+    const filePaths = globby
+      .sync(patterns, { dot: true })
+      .map(
+        filePath =>
+          path.isAbsolute(filePath)
+            ? path.relative(process.cwd(), filePath)
+            : filePath
+      );
     if (filePaths.length === 0) {
       console.error(`No matching files. Patterns tried: ${patterns.join(" ")}`);
       process.exitCode = 2;

tests_integration/__tests__/__snapshots__/ignore-absolute-path.js.snap

@@ -0,0 +1,6 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`support absolute filename 1`] = `
+"regular-module.js
+"
+`;

tests_integration/__tests__/ignore-absolute-path.js

@@ -0,0 +1,16 @@
+"use strict";
+
+const path = require("path");
+const runPrettier = require("../runPrettier");
+
+test("support absolute filename", () => {
+  const result = runPrettier("cli/ignore-absolute-path", [
+    path.resolve(__dirname, "../cli/ignore-absolute-path/ignored/module.js"),
+    path.resolve(__dirname, "../cli/ignore-absolute-path/depth1/ignored/*.js"),
+    path.resolve(__dirname, "../cli/ignore-absolute-path/regular-module.js"),
+    "-l"
+  ]);
+
+  expect(result.stdout).toMatchSnapshot();
+  expect(result.status).toEqual(1);
+});

tests_integration/cli/ignore-absolute-path/.prettierignore

@@ -0,0 +1,2 @@
+/ignored/
+depth1/ignored/

tests_integration/cli/ignore-absolute-path/depth1/ignored/module.js

@@ -0,0 +1 @@
+'use strict';

tests_integration/cli/ignore-absolute-path/ignored/module.js

@@ -0,0 +1 @@
+'use strict';

tests_integration/cli/ignore-absolute-path/regular-module.js

@@ -0,0 +1 @@
+'use strict';