Fix: Ignore inline plugin rule config in autoconfig (fixes #7860)

eslint · Jan 14, 2017 · e8cff13 · e8cff13
1 parent 6448ba0
commit e8cff13
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/lib/config/autoconfig.js b/lib/config/autoconfig.js
@@ -296,8 +296,13 @@ class Registry {
 
                     // It is possible that the error is from a configuration comment
                     // in a linted file, in which case there may not be a config
-                    // set in this ruleSetIdx. (https://github.com/eslint/eslint/issues/5992)
-                    if (lintedRegistry.rules[result.ruleId][ruleSetIdx]) {
+                    // set in this ruleSetIdx.
+                    // (https://github.com/eslint/eslint/issues/5992)
+                    // (https://github.com/eslint/eslint/issues/7860)
+                    if (
+                        lintedRegistry.rules[result.ruleId]
+                        && lintedRegistry.rules[result.ruleId][ruleSetIdx]
+                    ) {
                         lintedRegistry.rules[result.ruleId][ruleSetIdx].errorCount += 1;
                     }
                 });

diff --git a/tests/fixtures/autoconfig/source-with-comments.js b/tests/fixtures/autoconfig/source-with-comments.js
@@ -1,4 +1,5 @@
 /* eslint semi: [2, "never"] */
+/* eslint import/extensions: 2 */
 
 var foo = 42;
 var baz = "baz";