Option2 - no-extraneous-dependencies includes only paths that can be …

…resolved, but ignores import/ignore setting

Only resolved paths can be checked whether they actually live in "node_modules" directory
import/ignore does not make sense here, as it's mostly used to ignore imports unknown to this plugin, like .css files.
But still, you may import a css file from a dependency, and this rule should check that.

docs/rules/no-extraneous-dependencies.md

@@ -3,6 +3,8 @@
 Forbid the import of external modules that are not declared in the `package.json`'s `dependencies`, `devDependencies`, `optionalDependencies` or `peerDependencies`.
 The closest parent `package.json` will be used. If no `package.json` is found, the rule will not lint anything. This behaviour can be changed with the rule option `packageDir`.
 
+Modules have to be installed for this rule to work.
+
 ### Options
 
 This rule supports the following options:
@@ -33,10 +35,6 @@ Also there is one more option called `packageDir`, this option is to specify the
 "import/no-extraneous-dependencies": ["error", {"packageDir": './some-dir/'}]
 ```
 
-A module path that is [ignored] or couldn't be resolved will not be reported when imported.
-
-[ignored]: ../README.md#importignore
-
 ## Rule Details
 
 Given the following `package.json`:

src/rules/no-extraneous-dependencies.js

@@ -2,7 +2,6 @@ import path from 'path'
 import fs from 'fs'
 import readPkgUp from 'read-pkg-up'
 import minimatch from 'minimatch'
-import isIgnored from 'eslint-module-utils/ignore'
 import resolve from 'eslint-module-utils/resolve'
 import importType from '../core/importType'
 import isStaticRequire from '../core/staticRequire'
@@ -60,8 +59,7 @@ function reportIfMissing(context, deps, depsOptions, node, name) {
     return
   }
   const resolved = resolve(name, context)
-
-  if (!resolved || isIgnored(resolved, context)) {
+  if (!resolved) {
     return
   }
   const splitName = name.split('/')

tests/src/rules/no-extraneous-dependencies.js

@@ -69,14 +69,6 @@ ruleTester.run('no-extraneous-dependencies', rule, {
       code: 'import "doctrine"',
       options: [{packageDir: path.join(__dirname, '../../../')}],
     }),
-    test({
-      code: 'import bar from "not-a-dependency"',
-      settings: { 'import/ignore': ['/node_modules/'] },
-    }),
-    test({
-      code: 'import bar from "chai"',
-      settings: { 'import/ignore': ['/node_modules/'] },
-    }),
   ],
   invalid: [
     test({