docs/src/use/migrating-to-2.0.0.md
ESLint v2.0.0 is the second major version release. As a result, there are some significant changes between how ESLint worked during its life in 0.x and 1.x and how it will work going forward. These changes are the direct result of feedback from the ESLint community of users and were not made without due consideration for the upgrade path. We believe that these changes make ESLint even better, and while some work is necessary to upgrade, we hope the pain of this upgrade is small enough that you will see the benefit of upgrading.
Important: If you are upgrading from 0.x, please refer to Migrating to 1.0.0 as your starting point.
Due to a quirk in the way rule schemas worked, it was possible that you'd need to account for the rule severity (0, 1, or 2) in a rule schema if the options were sufficiently complex. That would result in a schema such as:
module.exports = {
type: "array",
items: [
{
enum: [0, 1, 2],
},
{
enum: ["always", "never"],
},
],
minItems: 1,
maxItems: 2,
};
This was confusing to rule developers as it seemed that rules shouldn't be in charge of validating their own severity. In 2.0.0, rules no longer need to check their own severity.
To address: If you are exporting a rule schema that checks severity, you need to make several changes:
minItems from 1 to 0maxItems by subtracting 1Here's what the schema from above looks like when properly converted:
module.exports = {
type: "array",
items: [
{
enum: ["always", "never"],
},
],
minItems: 0,
maxItems: 1,
};
The following rules have been deprecated with new rules created to take their place. The following is a list of the removed rules and their replacements:
no-arrow-condition.{"allowLoop": true, "allowSwitch": true} option.To address: You'll need to update your rule configurations to use the new rules. ESLint v2.0.0 will also warn you when you're using a rule that has been removed and will suggest the replacement rules. Hopefully, this will result in few surprises during the upgrade process.
Prior to 2.0.0, if a directory contained both an .eslintrc file and a package.json file with ESLint configuration information, the settings from the two files would be merged together. In 2.0.0, only the settings from the .eslintrc.* file are used and the ones in package.json are ignored when both are present. Otherwise, package.json can still be used with ESLint configuration, but only if no other .eslintrc.* files are present.
To address: If you have both an .eslintrc.* and package.json with ESLint configuration information in the same directory, combine your configurations into just one of those files.
Prior to 2.0.0, new global variables that were standardized as part of ES6 such as Promise, Map, Set, and Symbol were included in the built-in global environment. This could lead to potential issues when, for example, no-undef permitted use of the Promise constructor even in ES5 code where promises are unavailable. In 2.0.0, the built-in environment only includes the standard ES5 global variables, and the new ES6 global variables have been moved to the es6 environment.
To address: If you are writing ES6 code, enable the es6 environment if you have not already done so:
// In your .eslintrc
{
env: {
es6: true;
}
}
// Or in a configuration comment
/*eslint-env es6*/
Prior to 2.0.0, the way to enable language options was by using ecmaFeatures in your configuration. In 2.0.0:
ecmaFeatures property is now under a top-level parserOptions property.ecmaFeatures flags have been removed in favor of a ecmaVersion property under parserOptions that can be set to 3, 5 (default), or 6.ecmaFeatures.modules flag has been replaced by a sourceType property under parserOptions which can be set to "script" (default) or "module" for ES6 modules.To address: If you are using any ECMAScript 6 feature flags in ecmaFeatures, you'll need to use ecmaVersion: 6 instead. The ECMAScript 6 feature flags are:
arrowFunctions - enable arrow functionsbinaryLiterals - enable binary literalsblockBindings - enable let and const (aka block bindings)classes - enable classesdefaultParams - enable default function parametersdestructuring - enable destructuringforOf - enable for-of loopsgenerators - enable generatorsmodules - enable modules and global strict modeobjectLiteralComputedProperties - enable computed object literal property namesobjectLiteralDuplicateProperties - enable duplicate object literal properties in strict modeobjectLiteralShorthandMethods - enable object literal shorthand methodsobjectLiteralShorthandProperties - enable object literal shorthand propertiesoctalLiterals - enable octal literalsregexUFlag - enable the regular expression u flagregexYFlag - enable the regular expression y flagrestParams - enable the rest parametersspread - enable the spread operator for arrayssuperInFunctions - enable super references inside of functionstemplateStrings - enable template stringsunicodeCodePointEscapes - enable code point escapesIf you're using any of these flags, such as:
{
ecmaFeatures: {
arrowFunctions: true;
}
}
Then you should enable ES6 using ecmaVersion:
{
parserOptions: {
ecmaVersion: 6;
}
}
If you're using any non-ES6 flags in ecmaFeatures, you need to move those inside of parserOptions. For instance:
{
ecmaFeatures: {
jsx: true;
}
}
Then you should move ecmaFeatures under parserOptions:
{
parserOptions: {
ecmaFeatures: {
jsx: true;
}
}
}
If you were using ecmaFeatures.modules to enable ES6 module support like this:
{
ecmaFeatures: {
modules: true;
}
}
{
parserOptions: {
sourceType: "module";
}
}
Additionally, if you are using context.ecmaFeatures inside of your rules, then you'll need to update your code in the following ways:
context.ecmaFeatures.blockBindings, rewrite to check for context.parserOptions.ecmaVersion > 5.context.ecmaFeatures.modules, rewrite to check that the sourceType property of the Program node is "module".context.ecmaFeatures.jsx, rewrite to check for context.parserOptions.ecmaFeatures.jsx.If you have a plugin with rules and you are using RuleTester, then you also need to update the options you pass for rules that use ecmaFeatures. For example:
var ruleTester = new RuleTester();
ruleTester.run("no-var", rule, {
valid: [
{
code: "let x;",
parserOptions: { ecmaVersion: 6 },
},
],
});
If you're not using ecmaFeatures in your configuration or your custom/plugin rules and tests, then no change is needed.
"eslint:recommended"{
"extends": "eslint:recommended"
}
In 2.0.0, the following 11 rules were added to "eslint:recommended".
To address: If you don't want to be notified by those rules, you can simply disable those rules.
{
"extends": "eslint:recommended",
"rules": {
"no-case-declarations": 0,
"no-class-assign": 0,
"no-const-assign": 0,
"no-dupe-class-members": 0,
"no-empty-pattern": 0,
"no-new-symbol": 0,
"no-self-assign": 0,
"no-this-before-super": 0,
"no-unexpected-multiline": 0,
"no-unused-labels": 0,
"constructor-super": 0
}
}
We found some bugs in our scope analysis that needed to be addressed. Specifically, we were not properly accounting for global variables in all the ways they are defined.
Originally, Variable objects and Reference objects refer each other:
Variable#references property is an array of Reference objects which are referencing the variable.Reference#resolved property is a Variable object which are referenced.But until 1.x, the following variables and references had the wrong value (empty) in those properties:
var declarations in the global.function declarations in the global./* global */ comments.Now, those variables and references have correct values in these properties.
Scope#through property has references where Reference#resolved is null. So as a result of this change, the value of Scope#through property was changed also.
To address: If you are using Scope#through to find references of a built-in global variable, you need to make several changes.
For example, this is how you might locate the window global variable in 1.x:
var globalScope = context.getScope();
globalScope.through.forEach(function (reference) {
if (reference.identifier.name === "window") {
checkForWindow(reference);
}
});
This was a roundabout way to find the variable because it was added after the fact by ESLint. The window variable was in Scope#through because the definition couldn't be found.
In 2.0.0, window is no longer located in Scope#through because we have added back the correct declaration. That means you can reference the window object (or any other global object) directly. So the previous example would change to this:
var globalScope = context.getScope();
var variable = globalScope.set.get("window");
if (variable) {
variable.references.forEach(checkForWindow);
}
Further Reading: https://estools.github.io/escope/
eslint:recommendedThis will affect you if you are extending from eslint:recommended, and are enabling no-multiple-empty-lines or func-style with only a severity, such as:
{
"extends": "eslint:recommended",
"rules": {
"no-multiple-empty-lines": 2,
"func-style": 2
}
}
The rule no-multiple-empty-lines has no default exceptions, but in ESLint 1.x, a default from eslint:recommended was applied such that a maximum of two empty lines would be permitted.
The rule func-style has a default configuration of "expression", but in ESLint 1.x, eslint:recommended defaulted it to "declaration".
ESLint 2.0.0 removes these conflicting defaults, and so you may begin seeing linting errors related to these rules.
To address: If you would like to maintain the previous behavior, update your configuration for no-multiple-empty-lines by adding {"max": 2}, and change func-style to "declaration". For example:
{
"extends": "eslint:recommended",
"rules": {
"no-multiple-empty-lines": [2, { "max": 2 }],
"func-style": [2, "declaration"]
}
}
SourceCode constructor got to handle Unicode BOM.
If the first argument text has BOM, SourceCode constructor sets true to this.hasBOM and strips BOM from the text.
var SourceCode = require("eslint").SourceCode;
var code = new SourceCode("\uFEFFvar foo = bar;", ast);
assert(code.hasBOM === true);
assert(code.text === "var foo = bar;");
So the second argument ast also should be parsed from stripped text.
To address: If you are using SourceCode constructor in your code, please parse the source code after it stripped BOM:
var ast = yourParser.parse(text.replace(/^\uFEFF/, ""), options);
var sourceCode = new SourceCode(text, ast);
strict - defaults to "safe" (previous default was "function")Prior to v2.0.0, plugins could specify a rulesConfig for the plugin. The rulesConfig would automatically be applied whenever someone uses the plugin, which is the opposite of what ESLint does in every other situation (where nothing is on by default). To bring plugins behavior in line, we have removed support for rulesConfig in plugins.
To address: If you are using a plugin in your configuration file, you will need to manually enable the plugin rules in the configuration file.