Fix

package.json

@@ -49,7 +49,7 @@
     "html-element-attributes": "2.2.1",
     "html-styles": "1.0.0",
     "html-tag-names": "1.1.5",
-    "ignore": "5.1.6",
+    "ignore": "5.1.8",
     "jest-docblock": "26.0.0",
     "json-stable-stringify": "1.0.1",
     "leven": "3.1.0",

src/cli/util.js

@@ -4,7 +4,7 @@ const path = require("path");
 const camelCase = require("camelcase");
 const dashify = require("dashify");
 const fs = require("fs");
-
+const { isPathValid } = require("ignore");
 const chalk = require("chalk");
 const readline = require("readline");
 const stringify = require("json-stable-stringify");
@@ -379,7 +379,7 @@ function formatStdin(context) {
     .then((input) => {
       if (
         relativeFilepath &&
-        ignorer.ignores(fixWindowsSlashes(relativeFilepath))
+        isIgnored(ignorer, fixWindowsSlashes(relativeFilepath))
       ) {
         writeOutput(context, { formatted: input });
         return;
@@ -410,6 +410,10 @@ function createIgnorerFromContextOrDie(context) {
   }
 }
 
+function isIgnored(ignorer, file) {
+  return isPathValid(file) && ignorer.ignores(file);
+}
+
 function formatFiles(context) {
   // The ignorer will be used to filter file paths after the glob is checked,
   // before any files are actually written
@@ -436,7 +440,7 @@ function formatFiles(context) {
       ? path.relative(path.dirname(context.argv["ignore-path"]), filename)
       : filename;
 
-    const fileIgnored = ignorer.ignores(fixWindowsSlashes(ignoreFilename));
+    const fileIgnored = isIgnored(ignorer, fixWindowsSlashes(ignoreFilename));
     if (
       fileIgnored &&
       (context.argv["debug-check"] ||

src/common/get-file-info.js

@@ -1,5 +1,6 @@
 "use strict";
 
+const ignore = require("ignore");
 const createIgnorer = require("./create-ignorer");
 const options = require("../main/options");
 const config = require("../config/resolve-config");
@@ -29,9 +30,10 @@ async function getFileInfo(filePath, opts) {
   const ignorer = await createIgnorer(opts.ignorePath, opts.withNodeModules);
   return _getFileInfo({
     ignorer,
-    filePath: normalizeFilePath(filePath, opts.ignorePath),
+    filePath,
     plugins: opts.plugins,
     resolveConfig: opts.resolveConfig,
+    ignorePath: opts.ignorePath,
     sync: false,
   });
 }
@@ -51,9 +53,10 @@ getFileInfo.sync = function (filePath, opts) {
   const ignorer = createIgnorer.sync(opts.ignorePath, opts.withNodeModules);
   return _getFileInfo({
     ignorer,
-    filePath: normalizeFilePath(filePath, opts.ignorePath),
+    filePath,
     plugins: opts.plugins,
     resolveConfig: opts.resolveConfig,
+    ignorePath: opts.ignorePath,
     sync: true,
   });
 };
@@ -63,17 +66,12 @@ function _getFileInfo({
   filePath,
   plugins,
   resolveConfig = false,
+  ignorePath,
   sync = false,
 }) {
-  let ignored = true;
-  try {
-    ignored = ignorer.ignores(path.relative(process.cwd(), filePath));
-  } catch (_) {
-    // Invalid path
-  }
-
+  const relativeFilePath = normalizeFilePath(filePath, ignorePath)
   const fileInfo = {
-    ignored,
+    ignored: ignore.isPathValid(relativeFilePath) && ignorer.ignores(relativeFilePath),
     inferredParser: options.inferParser(filePath, plugins) || null,
   };
 
@@ -98,9 +96,10 @@ function _getFileInfo({
 }
 
 function normalizeFilePath(filePath, ignorePath) {
-  return ignorePath
-    ? path.relative(path.dirname(ignorePath), filePath)
-    : filePath;
+  return path.relative(
+    ignorePath ? path.dirname(ignorePath) : process.cwd(),
+    filePath
+  );
 }
 
 module.exports = getFileInfo;

tests_integration/__tests__/__snapshots__/file-info.js.snap

@@ -1,5 +1,15 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`Should not ignore files in parent dir (stderr) 1`] = `""`;
+
+exports[`Should not ignore files in parent dir (stdout) 1`] = `
+"{ \\"ignored\\": false, \\"inferredParser\\": \\"babel\\" }
+
+"
+`;
+
+exports[`Should not ignore files in parent dir (write) 1`] = `Array []`;
+
 exports[`extracts file-info for a file in not_node_modules (stderr) 1`] = `""`;
 
 exports[`extracts file-info for a file in not_node_modules (stdout) 1`] = `

tests_integration/__tests__/file-info.js

@@ -34,6 +34,16 @@ describe("extracts file-info with ignored=true for a file in .prettierignore", (
 });
 
 describe("extracts file-info with ignored=true for a file in a hand-picked .prettierignore", () => {
+  runPrettier("cli/ignore-path/", [
+    "--file-info",
+    "regular-module.js",
+    "--ignore-path=.prettierignore",
+  ]).test({
+    status: 0,
+  });
+});
+
+describe("Should not ignore files in parent dir", () => {
   runPrettier("cli/", [
     "--file-info",
     "regular-module.js",

yarn.lock

@@ -4133,10 +4133,10 @@ ignore-walk@3.0.3:
   dependencies:
     minimatch "^3.0.4"
 
-ignore@5.1.6:
-  version "5.1.6"
-  resolved "https://registry.yarnpkg.com/ignore/-/ignore-5.1.6.tgz#643194ad4bf2712f37852e386b6998eff0db2106"
-  integrity sha512-cgXgkypZBcCnOgSihyeqbo6gjIaIyDqPQB7Ra4vhE9m6kigdGoQDMHjviFhRZo3IMlRy6yElosoviMs5YxZXUA==
+ignore@5.1.8:
+  version "5.1.8"
+  resolved "https://registry.yarnpkg.com/ignore/-/ignore-5.1.8.tgz#f150a8b50a34289b33e22f5889abd4d8016f0e57"
+  integrity sha512-BMpfD7PpiETpBl/A6S498BaIJ6Y/ABT93ETbby2fP00v4EbvPBXWEoaR1UBPKs3iR53pJY7EtZk5KACI57i1Uw==
 
 ignore@^4.0.6:
   version "4.0.6"