fix: when lint default paths, don't try to lint non-existent or ignored paths

fixes #3167

Author: haoqunjiang

packages/@vue/cli-plugin-eslint/__tests__/eslintPlugin.spec.js

@@ -120,3 +120,22 @@ test('should not fix with --no-fix option', async () => {
   // files should not have been fixed
   expect(await read('src/main.js')).not.toMatch(';')
 })
+
+// #3167
+test('should not throw when src folder is ignored by .eslintignore', async () => {
+  const project = await create('eslint-ignore', {
+    plugins: {
+      '@vue/cli-plugin-babel': {},
+      '@vue/cli-plugin-eslint': {
+        config: 'airbnb',
+        lintOn: 'commit'
+      }
+    }
+  })
+
+  const { write, run } = project
+  await write('.eslintignore', 'src\n')
+
+  // should not throw
+  await run('vue-cli-service lint')
+})

packages/@vue/cli-plugin-eslint/lint.js

@@ -31,17 +31,27 @@ module.exports = function lint (args = {}, api) {
     cwd
   }, argsConfig)
 
+  const engine = new CLIEngine(config)
+
+  // .eslintrc.js files (ignored by default)
+  const dotFiles = [
+    '.*.js',
+    '{src,tests}/**/.*.js'
+  ].filter(pattern => globby.sync(path.join(cwd, pattern)).length)
+
   const defaultFilesToLint = [
     'src',
     'tests',
     // root config files
-    '*.js',
-    // .eslintrc files (ignored by default)
-    '.*.js',
-    '{src,tests}/**/.*.js'
-  ].filter(pattern => globby.sync(path.join(cwd, pattern)).length)
+    '*.js'
+  ]
+    .filter(pattern =>
+      globby
+        .sync(path.join(cwd, pattern))
+        .some(p => !engine.isPathIgnored(p))
+    )
+    .concat(dotFiles)
 
-  const engine = new CLIEngine(config)
   const files = args._ && args._.length
     ? args._
     : defaultFilesToLint