Chore: avoid applying eslint-env comments twice (#9278)

eslint · Oct 14, 2017 · db41408 · db41408
1 parent febb897
commit db41408
Showing 1 changed file with 1 addition and 13 deletions.
diff --git a/lib/linter.js b/lib/linter.js
@@ -291,7 +291,7 @@ function createDisableDirectives(type, loc, value) {
  */
 function modifyConfigsFromComments(filename, ast, config, linterContext) {
 
-    let commentConfig = {
+    const commentConfig = {
         exported: {},
         astGlobals: {},
         rules: {},
@@ -320,10 +320,6 @@ function modifyConfigsFromComments(filename, ast, config, linterContext) {
                         Object.assign(commentConfig.astGlobals, parseBooleanConfig(value, comment));
                         break;
 
-                    case "eslint-env":
-                        Object.assign(commentConfig.env, parseListConfig(value));
-                        break;
-
                     case "eslint-disable":
                         [].push.apply(disableDirectives, createDisableDirectives("disable", comment.loc.start, value));
                         break;
@@ -361,14 +357,6 @@ function modifyConfigsFromComments(filename, ast, config, linterContext) {
         }
     });
 
-    // apply environment configs
-    Object.keys(commentConfig.env).forEach(name => {
-        const env = linterContext.environments.get(name);
-
-        if (env) {
-            commentConfig = ConfigOps.merge(commentConfig, env);
-        }
-    });
     Object.assign(commentConfig.rules, commentRules);
 
     return {