New: WIP add eslintIgnore support to package.json (fixes eslint#8458)

More tests required
VictorHom · Jun 6, 2017 · e136502 · e136502
1 parent 54449a4
commit e136502
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/lib/ignored-paths.js b/lib/ignored-paths.js
@@ -183,9 +183,10 @@ class IgnoredPaths {
                         const configOptions = config.load(packageJSONPath, configContext);
 
                         if (configOptions.eslintIgnore) {
-                            const filesToIgnore = configOptions.eslintIgnore;
-
-                            debug(filesToIgnore);
+                            configOptions.eslintIgnore.forEach( pattern => {
+                                addPattern(this.ig.custom, pattern);
+                                addPattern(this.ig.default, pattern);
+                            });
                         }
                     }
                 } catch (e) {

diff --git a/tests/lib/ignored-paths.js b/tests/lib/ignored-paths.js
@@ -165,9 +165,12 @@ describe("IgnoredPaths", () => {
             );
         });
 
-        it("shoud load package.json's array of files to ignore in eslintIgnore", () => {
+        it("should load package.json's array of files to ignore in eslintIgnore", () => {
             const expectedIgnoreFile = getFixturePath("package-json-ignore/package.json");
             const ignoredPaths = new IgnoredPaths({ ignore: true, cwd: getFixturePath("package-json-ignore") });
+
+            assert.isTrue(ignoredPaths.contains("hello.js"));
+            assert.isTrue(ignoredPaths.contains("world.js"));
         });
     });