-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Closed
Labels
acceptedThere is consensus among the team that this change meets the criteria for inclusionThere is consensus among the team that this change meets the criteria for inclusionarchived due to ageThis issue has been archived; please open a new issue for any further discussionThis issue has been archived; please open a new issue for any further discussioncoreRelates to ESLint's core APIs and featuresRelates to ESLint's core APIs and featuresenhancementThis change enhances an existing feature of ESLintThis change enhances an existing feature of ESLint
Description
The version of ESLint you are using:
- 3.1.1
The problem you want to solve:
We have several accepted issues we will deprecate rules. (#6080, #6586)
We generate eslint:all
preset from rule files automatically, and this would include deprecated rules.
I think eslint:all
preset should not include deprecated rules.
Your take on the correct solution to problem:
- It adds
deprecated
information into themeta
property of rules. eslint:all
checks themeta.deprecated
property.
var enabledRules = ruleFiles.reduce(function(result, filename) {
const ruleId = path.basename(filename, ".js");
if (path.extname(filename) === ".js" && !rules.get(ruleId).meta.deprecated) {
result[ruleId] = "error";
}
return result;
}, {});
gyandeeps
Metadata
Metadata
Assignees
Labels
acceptedThere is consensus among the team that this change meets the criteria for inclusionThere is consensus among the team that this change meets the criteria for inclusionarchived due to ageThis issue has been archived; please open a new issue for any further discussionThis issue has been archived; please open a new issue for any further discussioncoreRelates to ESLint's core APIs and featuresRelates to ESLint's core APIs and featuresenhancementThis change enhances an existing feature of ESLintThis change enhances an existing feature of ESLint