diff --git a/Makefile.js b/Makefile.js index 42e8ed03b64..31c8b3d1706 100644 --- a/Makefile.js +++ b/Makefile.js @@ -177,8 +177,8 @@ function generateFormatterExamples(formatterInfo, prereleaseVersion) { */ function generateRuleIndexPage() { const outputFile = "../website/_data/rules.yml", - categoryList = "conf/category-list.json", - categoriesData = JSON.parse(cat(path.resolve(categoryList))); + ruleTypes = "conf/rule-type-list.json", + ruleTypesData = JSON.parse(cat(path.resolve(ruleTypes))); RULE_FILES .map(filename => [filename, path.basename(filename, ".js")]) @@ -189,7 +189,7 @@ function generateRuleIndexPage() { const rule = require(path.resolve(filename)); if (rule.meta.deprecated) { - categoriesData.deprecated.rules.push({ + ruleTypesData.deprecated.rules.push({ name: basename, replacedBy: rule.meta.replacedBy || [] }); @@ -201,20 +201,20 @@ function generateRuleIndexPage() { fixable: !!rule.meta.fixable, hasSuggestions: !!rule.meta.hasSuggestions }, - category = categoriesData.categories.find(c => c.name === rule.meta.docs.category); + ruleType = ruleTypesData.types.find(c => c.name === rule.meta.type); - if (!category.rules) { - category.rules = []; + if (!ruleType.rules) { + ruleType.rules = []; } - category.rules.push(output); + ruleType.rules.push(output); } }); // `.rules` will be `undefined` if all rules in category are deprecated. - categoriesData.categories = categoriesData.categories.filter(category => !!category.rules); + ruleTypesData.types = ruleTypesData.types.filter(ruleType => !!ruleType.rules); - const output = yaml.dump(categoriesData, { sortKeys: true }); + const output = yaml.dump(ruleTypesData, { sortKeys: true }); output.to(outputFile); } diff --git a/conf/category-list.json b/conf/rule-type-list.json similarity index 74% rename from conf/category-list.json rename to conf/rule-type-list.json index cd3b816b657..f362aa412f9 100644 --- a/conf/category-list.json +++ b/conf/rule-type-list.json @@ -1,11 +1,8 @@ { - "categories": [ - { "name": "Possible Errors", "description": "These rules relate to possible syntax or logic errors in JavaScript code:" }, - { "name": "Best Practices", "description": "These rules relate to better ways of doing things to help you avoid problems:" }, - { "name": "Strict Mode", "description": "These rules relate to strict mode directives:" }, - { "name": "Variables", "description": "These rules relate to variable declarations:" }, - { "name": "Stylistic Issues", "description": "These rules relate to style guidelines, and are therefore quite subjective:" }, - { "name": "ECMAScript 6", "description": "These rules relate to ES6, also known as ES2015:" } + "types": [ + { "name": "problem", "displayName": "Possible Problems", "description": "These rules relate to possible logic errors in code:" }, + { "name": "suggestion", "displayName": "Suggestions", "description": "These rules suggest alternate ways of doing things:" }, + { "name": "layout", "displayName": "Layout & Formatting", "description": "These rules care about how the code looks rather than how it executes:" } ], "deprecated": { "name": "Deprecated", diff --git a/lib/rules/accessor-pairs.js b/lib/rules/accessor-pairs.js index 0e0d07a00c9..f0472526680 100644 --- a/lib/rules/accessor-pairs.js +++ b/lib/rules/accessor-pairs.js @@ -140,7 +140,6 @@ module.exports = { docs: { description: "enforce getter and setter pairs in objects and classes", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/accessor-pairs" }, diff --git a/lib/rules/array-bracket-newline.js b/lib/rules/array-bracket-newline.js index b4b4dd430f6..28a05b35043 100644 --- a/lib/rules/array-bracket-newline.js +++ b/lib/rules/array-bracket-newline.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "enforce linebreaks after opening and before closing array brackets", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/array-bracket-newline" }, diff --git a/lib/rules/array-bracket-spacing.js b/lib/rules/array-bracket-spacing.js index c2b77a641f7..1eea99c14ca 100644 --- a/lib/rules/array-bracket-spacing.js +++ b/lib/rules/array-bracket-spacing.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce consistent spacing inside array brackets", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/array-bracket-spacing" }, diff --git a/lib/rules/array-callback-return.js b/lib/rules/array-callback-return.js index 7267347149d..d13ecd71790 100644 --- a/lib/rules/array-callback-return.js +++ b/lib/rules/array-callback-return.js @@ -139,7 +139,6 @@ module.exports = { docs: { description: "enforce `return` statements in callbacks of array methods", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/array-callback-return" }, diff --git a/lib/rules/array-element-newline.js b/lib/rules/array-element-newline.js index b7a967865b9..535fa2155a2 100644 --- a/lib/rules/array-element-newline.js +++ b/lib/rules/array-element-newline.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "enforce line breaks after each array element", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/array-element-newline" }, diff --git a/lib/rules/arrow-body-style.js b/lib/rules/arrow-body-style.js index 5b8a5f01167..3a3f544444e 100644 --- a/lib/rules/arrow-body-style.js +++ b/lib/rules/arrow-body-style.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "require braces around arrow function bodies", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/arrow-body-style" }, diff --git a/lib/rules/arrow-parens.js b/lib/rules/arrow-parens.js index eaa1aab0238..4f4dea0e237 100644 --- a/lib/rules/arrow-parens.js +++ b/lib/rules/arrow-parens.js @@ -33,7 +33,6 @@ module.exports = { docs: { description: "require parentheses around arrow function arguments", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/arrow-parens" }, diff --git a/lib/rules/arrow-spacing.js b/lib/rules/arrow-spacing.js index e5110c6c87d..9e1ed71d4bd 100644 --- a/lib/rules/arrow-spacing.js +++ b/lib/rules/arrow-spacing.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "enforce consistent spacing before and after the arrow in arrow functions", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/arrow-spacing" }, diff --git a/lib/rules/block-scoped-var.js b/lib/rules/block-scoped-var.js index 481057ba6c7..10125e61fde 100644 --- a/lib/rules/block-scoped-var.js +++ b/lib/rules/block-scoped-var.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "enforce the use of variables within the scope they are defined", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/block-scoped-var" }, diff --git a/lib/rules/block-spacing.js b/lib/rules/block-spacing.js index c4b30b0b70b..13cfbf0e2cb 100644 --- a/lib/rules/block-spacing.js +++ b/lib/rules/block-spacing.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow or enforce spaces inside of blocks after opening block and before closing block", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/block-spacing" }, diff --git a/lib/rules/brace-style.js b/lib/rules/brace-style.js index 07223d10d4a..60aa5326935 100644 --- a/lib/rules/brace-style.js +++ b/lib/rules/brace-style.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "enforce consistent brace style for blocks", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/brace-style" }, diff --git a/lib/rules/callback-return.js b/lib/rules/callback-return.js index fa66e6383b7..9cc9089fbff 100644 --- a/lib/rules/callback-return.js +++ b/lib/rules/callback-return.js @@ -18,7 +18,6 @@ module.exports = { docs: { description: "require `return` statements after callbacks", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/callback-return" }, diff --git a/lib/rules/camelcase.js b/lib/rules/camelcase.js index 52d44deb994..7e8fc68da63 100644 --- a/lib/rules/camelcase.js +++ b/lib/rules/camelcase.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce camelcase naming convention", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/camelcase" }, diff --git a/lib/rules/capitalized-comments.js b/lib/rules/capitalized-comments.js index d7524b878d2..e5f429356b2 100644 --- a/lib/rules/capitalized-comments.js +++ b/lib/rules/capitalized-comments.js @@ -105,7 +105,6 @@ module.exports = { docs: { description: "enforce or disallow capitalization of the first letter of a comment", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/capitalized-comments" }, diff --git a/lib/rules/class-methods-use-this.js b/lib/rules/class-methods-use-this.js index a6e82dfb620..7e98f4bb739 100644 --- a/lib/rules/class-methods-use-this.js +++ b/lib/rules/class-methods-use-this.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "enforce that class methods utilize `this`", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/class-methods-use-this" }, diff --git a/lib/rules/comma-dangle.js b/lib/rules/comma-dangle.js index 327dca7f863..e97a59886e3 100644 --- a/lib/rules/comma-dangle.js +++ b/lib/rules/comma-dangle.js @@ -76,7 +76,6 @@ module.exports = { docs: { description: "require or disallow trailing commas", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/comma-dangle" }, diff --git a/lib/rules/comma-spacing.js b/lib/rules/comma-spacing.js index 2bf41a00bb6..d30a5ef320d 100644 --- a/lib/rules/comma-spacing.js +++ b/lib/rules/comma-spacing.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce consistent spacing before and after commas", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/comma-spacing" }, diff --git a/lib/rules/comma-style.js b/lib/rules/comma-style.js index 824ad89b2f9..1d62fcf1c4d 100644 --- a/lib/rules/comma-style.js +++ b/lib/rules/comma-style.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "enforce consistent comma style", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/comma-style" }, diff --git a/lib/rules/complexity.js b/lib/rules/complexity.js index 116c8ad0a63..e3e300386a6 100644 --- a/lib/rules/complexity.js +++ b/lib/rules/complexity.js @@ -23,7 +23,6 @@ module.exports = { docs: { description: "enforce a maximum cyclomatic complexity allowed in a program", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/complexity" }, diff --git a/lib/rules/computed-property-spacing.js b/lib/rules/computed-property-spacing.js index 9eb665c1794..c8d8834ea92 100644 --- a/lib/rules/computed-property-spacing.js +++ b/lib/rules/computed-property-spacing.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce consistent spacing inside computed property brackets", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/computed-property-spacing" }, diff --git a/lib/rules/consistent-return.js b/lib/rules/consistent-return.js index 0e20209af56..b509c36564f 100644 --- a/lib/rules/consistent-return.js +++ b/lib/rules/consistent-return.js @@ -46,7 +46,6 @@ module.exports = { docs: { description: "require `return` statements to either always or never specify values", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/consistent-return" }, diff --git a/lib/rules/consistent-this.js b/lib/rules/consistent-this.js index 75b30e8fc04..025f3d0a340 100644 --- a/lib/rules/consistent-this.js +++ b/lib/rules/consistent-this.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "enforce consistent naming when capturing the current execution context", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/consistent-this" }, diff --git a/lib/rules/constructor-super.js b/lib/rules/constructor-super.js index dfec18fb65a..38eb489327d 100644 --- a/lib/rules/constructor-super.js +++ b/lib/rules/constructor-super.js @@ -122,7 +122,6 @@ module.exports = { docs: { description: "require `super()` calls in constructors", - category: "ECMAScript 6", recommended: true, url: "https://eslint.org/docs/rules/constructor-super" }, diff --git a/lib/rules/curly.js b/lib/rules/curly.js index 61dcd8024bb..57c2e72e4e4 100644 --- a/lib/rules/curly.js +++ b/lib/rules/curly.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "enforce consistent brace style for all control statements", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/curly" }, diff --git a/lib/rules/default-case-last.js b/lib/rules/default-case-last.js index 80c5d6bda73..1eeadd1e716 100644 --- a/lib/rules/default-case-last.js +++ b/lib/rules/default-case-last.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce default clauses in switch statements to be last", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/default-case-last" }, diff --git a/lib/rules/default-case.js b/lib/rules/default-case.js index 023854153f9..b839aa20133 100644 --- a/lib/rules/default-case.js +++ b/lib/rules/default-case.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "require `default` cases in `switch` statements", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/default-case" }, diff --git a/lib/rules/default-param-last.js b/lib/rules/default-param-last.js index 84751cdf4fd..8382d46e760 100644 --- a/lib/rules/default-param-last.js +++ b/lib/rules/default-param-last.js @@ -11,7 +11,6 @@ module.exports = { docs: { description: "enforce default parameters to be last", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/default-param-last" }, diff --git a/lib/rules/dot-location.js b/lib/rules/dot-location.js index a8d5a760562..d80f87090a7 100644 --- a/lib/rules/dot-location.js +++ b/lib/rules/dot-location.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "enforce consistent newlines before and after dots", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/dot-location" }, diff --git a/lib/rules/dot-notation.js b/lib/rules/dot-notation.js index 20434f3594c..1cd908f7a2c 100644 --- a/lib/rules/dot-notation.js +++ b/lib/rules/dot-notation.js @@ -26,7 +26,6 @@ module.exports = { docs: { description: "enforce dot notation whenever possible", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/dot-notation" }, diff --git a/lib/rules/eol-last.js b/lib/rules/eol-last.js index c141d9befd2..f8b922c2503 100644 --- a/lib/rules/eol-last.js +++ b/lib/rules/eol-last.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "require or disallow newline at the end of files", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/eol-last" }, diff --git a/lib/rules/eqeqeq.js b/lib/rules/eqeqeq.js index 09fb3bc14b5..d3e6b5af0b6 100644 --- a/lib/rules/eqeqeq.js +++ b/lib/rules/eqeqeq.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "require the use of `===` and `!==`", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/eqeqeq" }, diff --git a/lib/rules/for-direction.js b/lib/rules/for-direction.js index c15d10e5f84..abe4ad3c6b8 100644 --- a/lib/rules/for-direction.js +++ b/lib/rules/for-direction.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce \"for\" loop update clause moving the counter in the right direction.", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/for-direction" }, diff --git a/lib/rules/func-call-spacing.js b/lib/rules/func-call-spacing.js index 132a5833143..a6ebde4ac0b 100644 --- a/lib/rules/func-call-spacing.js +++ b/lib/rules/func-call-spacing.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "require or disallow spacing between function identifiers and their invocations", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/func-call-spacing" }, diff --git a/lib/rules/func-name-matching.js b/lib/rules/func-name-matching.js index 755c2ee5075..c15aa4a3423 100644 --- a/lib/rules/func-name-matching.js +++ b/lib/rules/func-name-matching.js @@ -74,7 +74,6 @@ module.exports = { docs: { description: "require function names to match the name of the variable or property to which they are assigned", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/func-name-matching" }, diff --git a/lib/rules/func-names.js b/lib/rules/func-names.js index d6fd9e07199..589903c96a5 100644 --- a/lib/rules/func-names.js +++ b/lib/rules/func-names.js @@ -30,7 +30,6 @@ module.exports = { docs: { description: "require or disallow named `function` expressions", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/func-names" }, diff --git a/lib/rules/func-style.js b/lib/rules/func-style.js index e150b1a76f2..0921ff54cd4 100644 --- a/lib/rules/func-style.js +++ b/lib/rules/func-style.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "enforce the consistent use of either `function` declarations or expressions", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/func-style" }, diff --git a/lib/rules/function-call-argument-newline.js b/lib/rules/function-call-argument-newline.js index b6abbe95fa9..ed4e296fd6e 100644 --- a/lib/rules/function-call-argument-newline.js +++ b/lib/rules/function-call-argument-newline.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce line breaks between arguments of a function call", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/function-call-argument-newline" }, diff --git a/lib/rules/function-paren-newline.js b/lib/rules/function-paren-newline.js index 8884b9b97fe..18435b78978 100644 --- a/lib/rules/function-paren-newline.js +++ b/lib/rules/function-paren-newline.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "enforce consistent line breaks inside function parentheses", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/function-paren-newline" }, diff --git a/lib/rules/generator-star-spacing.js b/lib/rules/generator-star-spacing.js index 65534f727fa..c50445c9c9b 100644 --- a/lib/rules/generator-star-spacing.js +++ b/lib/rules/generator-star-spacing.js @@ -31,7 +31,6 @@ module.exports = { docs: { description: "enforce consistent spacing around `*` operators in generator functions", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/generator-star-spacing" }, diff --git a/lib/rules/getter-return.js b/lib/rules/getter-return.js index c54ebfb4ffb..8bb42536e8c 100644 --- a/lib/rules/getter-return.js +++ b/lib/rules/getter-return.js @@ -35,7 +35,6 @@ module.exports = { docs: { description: "enforce `return` statements in getters", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/getter-return" }, diff --git a/lib/rules/global-require.js b/lib/rules/global-require.js index 09d0332007e..aa8508eb566 100644 --- a/lib/rules/global-require.js +++ b/lib/rules/global-require.js @@ -57,7 +57,6 @@ module.exports = { docs: { description: "require `require()` calls to be placed at top-level module scope", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/global-require" }, diff --git a/lib/rules/grouped-accessor-pairs.js b/lib/rules/grouped-accessor-pairs.js index a790f83750b..cc4a4b522a1 100644 --- a/lib/rules/grouped-accessor-pairs.js +++ b/lib/rules/grouped-accessor-pairs.js @@ -96,7 +96,6 @@ module.exports = { docs: { description: "require grouped accessor pairs in object literals and classes", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/grouped-accessor-pairs" }, diff --git a/lib/rules/guard-for-in.js b/lib/rules/guard-for-in.js index 2c0976d997b..6f877bab958 100644 --- a/lib/rules/guard-for-in.js +++ b/lib/rules/guard-for-in.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require `for-in` loops to include an `if` statement", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/guard-for-in" }, diff --git a/lib/rules/handle-callback-err.js b/lib/rules/handle-callback-err.js index b92490ad16b..2f782e7e8aa 100644 --- a/lib/rules/handle-callback-err.js +++ b/lib/rules/handle-callback-err.js @@ -19,7 +19,6 @@ module.exports = { docs: { description: "require error handling in callbacks", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/handle-callback-err" }, diff --git a/lib/rules/id-blacklist.js b/lib/rules/id-blacklist.js index 60f6e73ed05..3723651ba57 100644 --- a/lib/rules/id-blacklist.js +++ b/lib/rules/id-blacklist.js @@ -118,7 +118,6 @@ module.exports = { docs: { description: "disallow specified identifiers", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/id-blacklist" }, diff --git a/lib/rules/id-denylist.js b/lib/rules/id-denylist.js index ea9dfddd62e..2b346355423 100644 --- a/lib/rules/id-denylist.js +++ b/lib/rules/id-denylist.js @@ -99,7 +99,6 @@ module.exports = { docs: { description: "disallow specified identifiers", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/id-denylist" }, diff --git a/lib/rules/id-length.js b/lib/rules/id-length.js index 64c600f8343..ac6385f59c0 100644 --- a/lib/rules/id-length.js +++ b/lib/rules/id-length.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce minimum and maximum identifier lengths", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/id-length" }, diff --git a/lib/rules/id-match.js b/lib/rules/id-match.js index 46371a7a661..7a6cd058f2e 100644 --- a/lib/rules/id-match.js +++ b/lib/rules/id-match.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require identifiers to match a specified regular expression", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/id-match" }, diff --git a/lib/rules/implicit-arrow-linebreak.js b/lib/rules/implicit-arrow-linebreak.js index 409145e7dc1..2d09552440b 100644 --- a/lib/rules/implicit-arrow-linebreak.js +++ b/lib/rules/implicit-arrow-linebreak.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce the location of arrow function bodies", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/implicit-arrow-linebreak" }, diff --git a/lib/rules/indent-legacy.js b/lib/rules/indent-legacy.js index a26ee87b130..68bd9db99ed 100644 --- a/lib/rules/indent-legacy.js +++ b/lib/rules/indent-legacy.js @@ -25,7 +25,6 @@ module.exports = { docs: { description: "enforce consistent indentation", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/indent-legacy" }, diff --git a/lib/rules/indent.js b/lib/rules/indent.js index abd05c63de2..7b17e3e5402 100644 --- a/lib/rules/indent.js +++ b/lib/rules/indent.js @@ -499,7 +499,6 @@ module.exports = { docs: { description: "enforce consistent indentation", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/indent" }, diff --git a/lib/rules/init-declarations.js b/lib/rules/init-declarations.js index 6cfdf92c909..d994bbc1ea0 100644 --- a/lib/rules/init-declarations.js +++ b/lib/rules/init-declarations.js @@ -48,7 +48,6 @@ module.exports = { docs: { description: "require or disallow initialization in variable declarations", - category: "Variables", recommended: false, url: "https://eslint.org/docs/rules/init-declarations" }, diff --git a/lib/rules/jsx-quotes.js b/lib/rules/jsx-quotes.js index 3b282df2f07..cbadc19d5ed 100644 --- a/lib/rules/jsx-quotes.js +++ b/lib/rules/jsx-quotes.js @@ -42,7 +42,6 @@ module.exports = { docs: { description: "enforce the consistent use of either double or single quotes in JSX attributes", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/jsx-quotes" }, diff --git a/lib/rules/key-spacing.js b/lib/rules/key-spacing.js index 12b5ce8e10a..cb176dd00d8 100644 --- a/lib/rules/key-spacing.js +++ b/lib/rules/key-spacing.js @@ -139,7 +139,6 @@ module.exports = { docs: { description: "enforce consistent spacing between keys and values in object literal properties", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/key-spacing" }, diff --git a/lib/rules/keyword-spacing.js b/lib/rules/keyword-spacing.js index 458d15e1fdb..e9441ad1708 100644 --- a/lib/rules/keyword-spacing.js +++ b/lib/rules/keyword-spacing.js @@ -67,7 +67,6 @@ module.exports = { docs: { description: "enforce consistent spacing before and after keywords", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/keyword-spacing" }, diff --git a/lib/rules/line-comment-position.js b/lib/rules/line-comment-position.js index 77ee147cbec..ad109a4f795 100644 --- a/lib/rules/line-comment-position.js +++ b/lib/rules/line-comment-position.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce position of line comments", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/line-comment-position" }, diff --git a/lib/rules/linebreak-style.js b/lib/rules/linebreak-style.js index b3b393ead77..92996ebd331 100644 --- a/lib/rules/linebreak-style.js +++ b/lib/rules/linebreak-style.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "enforce consistent linebreak style", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/linebreak-style" }, diff --git a/lib/rules/lines-around-comment.js b/lib/rules/lines-around-comment.js index 6806e793cd1..79bcbb7fc19 100644 --- a/lib/rules/lines-around-comment.js +++ b/lib/rules/lines-around-comment.js @@ -55,7 +55,6 @@ module.exports = { docs: { description: "require empty lines around comments", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/lines-around-comment" }, diff --git a/lib/rules/lines-around-directive.js b/lib/rules/lines-around-directive.js index fb439dad6a3..39782aaf7ec 100644 --- a/lib/rules/lines-around-directive.js +++ b/lib/rules/lines-around-directive.js @@ -18,7 +18,6 @@ module.exports = { docs: { description: "require or disallow newlines around directives", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/lines-around-directive" }, diff --git a/lib/rules/lines-between-class-members.js b/lib/rules/lines-between-class-members.js index 97235303699..cb5c02df9cf 100644 --- a/lib/rules/lines-between-class-members.js +++ b/lib/rules/lines-between-class-members.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "require or disallow an empty line between class members", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/lines-between-class-members" }, diff --git a/lib/rules/max-classes-per-file.js b/lib/rules/max-classes-per-file.js index bb48a546e95..3f3eef6213c 100644 --- a/lib/rules/max-classes-per-file.js +++ b/lib/rules/max-classes-per-file.js @@ -19,7 +19,6 @@ module.exports = { docs: { description: "enforce a maximum number of classes per file", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/max-classes-per-file" }, diff --git a/lib/rules/max-depth.js b/lib/rules/max-depth.js index 5c5296bec00..415598b2929 100644 --- a/lib/rules/max-depth.js +++ b/lib/rules/max-depth.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce a maximum depth that blocks can be nested", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/max-depth" }, diff --git a/lib/rules/max-len.js b/lib/rules/max-len.js index 710fb8ede96..8c7985d3e16 100644 --- a/lib/rules/max-len.js +++ b/lib/rules/max-len.js @@ -69,7 +69,6 @@ module.exports = { docs: { description: "enforce a maximum line length", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/max-len" }, diff --git a/lib/rules/max-lines-per-function.js b/lib/rules/max-lines-per-function.js index 161a1c5bce7..b2130ca260b 100644 --- a/lib/rules/max-lines-per-function.js +++ b/lib/rules/max-lines-per-function.js @@ -71,7 +71,6 @@ module.exports = { docs: { description: "enforce a maximum number of lines of code in a function", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/max-lines-per-function" }, diff --git a/lib/rules/max-lines.js b/lib/rules/max-lines.js index 0837dd1f0fd..291d7d9c1a8 100644 --- a/lib/rules/max-lines.js +++ b/lib/rules/max-lines.js @@ -34,7 +34,6 @@ module.exports = { docs: { description: "enforce a maximum number of lines per file", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/max-lines" }, diff --git a/lib/rules/max-nested-callbacks.js b/lib/rules/max-nested-callbacks.js index df1baceeb41..df24a96da58 100644 --- a/lib/rules/max-nested-callbacks.js +++ b/lib/rules/max-nested-callbacks.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce a maximum depth that callbacks can be nested", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/max-nested-callbacks" }, diff --git a/lib/rules/max-params.js b/lib/rules/max-params.js index 8fb798401cb..c8be60e5774 100644 --- a/lib/rules/max-params.js +++ b/lib/rules/max-params.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "enforce a maximum number of parameters in function definitions", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/max-params" }, diff --git a/lib/rules/max-statements-per-line.js b/lib/rules/max-statements-per-line.js index 5407cff3c54..7c743292bd6 100644 --- a/lib/rules/max-statements-per-line.js +++ b/lib/rules/max-statements-per-line.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "enforce a maximum number of statements allowed per line", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/max-statements-per-line" }, diff --git a/lib/rules/max-statements.js b/lib/rules/max-statements.js index 65d5539550d..6f48c9218a9 100644 --- a/lib/rules/max-statements.js +++ b/lib/rules/max-statements.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "enforce a maximum number of statements allowed in function blocks", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/max-statements" }, diff --git a/lib/rules/multiline-comment-style.js b/lib/rules/multiline-comment-style.js index 9524818b8bd..da5ee50df4a 100644 --- a/lib/rules/multiline-comment-style.js +++ b/lib/rules/multiline-comment-style.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce a particular style for multiline comments", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/multiline-comment-style" }, diff --git a/lib/rules/multiline-ternary.js b/lib/rules/multiline-ternary.js index 98360b9cad4..6f468c828c0 100644 --- a/lib/rules/multiline-ternary.js +++ b/lib/rules/multiline-ternary.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "enforce newlines between operands of ternary expressions", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/multiline-ternary" }, diff --git a/lib/rules/new-cap.js b/lib/rules/new-cap.js index ab3a3674110..9abf3379b8a 100644 --- a/lib/rules/new-cap.js +++ b/lib/rules/new-cap.js @@ -82,7 +82,6 @@ module.exports = { docs: { description: "require constructor names to begin with a capital letter", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/new-cap" }, diff --git a/lib/rules/new-parens.js b/lib/rules/new-parens.js index 405ec1b515a..786300dba7d 100644 --- a/lib/rules/new-parens.js +++ b/lib/rules/new-parens.js @@ -25,7 +25,6 @@ module.exports = { docs: { description: "enforce or disallow parentheses when invoking a constructor with no arguments", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/new-parens" }, diff --git a/lib/rules/newline-after-var.js b/lib/rules/newline-after-var.js index 3c36e2bd0ac..05fb39493ab 100644 --- a/lib/rules/newline-after-var.js +++ b/lib/rules/newline-after-var.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "require or disallow an empty line after variable declarations", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/newline-after-var" }, diff --git a/lib/rules/newline-before-return.js b/lib/rules/newline-before-return.js index de94b5bea88..c2680c606dc 100644 --- a/lib/rules/newline-before-return.js +++ b/lib/rules/newline-before-return.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require an empty line before `return` statements", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/newline-before-return" }, diff --git a/lib/rules/newline-per-chained-call.js b/lib/rules/newline-per-chained-call.js index 7ae5c429c99..8de9a6a2398 100644 --- a/lib/rules/newline-per-chained-call.js +++ b/lib/rules/newline-per-chained-call.js @@ -18,7 +18,6 @@ module.exports = { docs: { description: "require a newline after each call in a method chain", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/newline-per-chained-call" }, diff --git a/lib/rules/no-alert.js b/lib/rules/no-alert.js index 702b4d2ba7c..918b98489a7 100644 --- a/lib/rules/no-alert.js +++ b/lib/rules/no-alert.js @@ -88,7 +88,6 @@ module.exports = { docs: { description: "disallow the use of `alert`, `confirm`, and `prompt`", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-alert" }, diff --git a/lib/rules/no-array-constructor.js b/lib/rules/no-array-constructor.js index 90c6d6bbd59..0904fa6d8f0 100644 --- a/lib/rules/no-array-constructor.js +++ b/lib/rules/no-array-constructor.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow `Array` constructors", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-array-constructor" }, diff --git a/lib/rules/no-async-promise-executor.js b/lib/rules/no-async-promise-executor.js index 553311e5804..27116f1da78 100644 --- a/lib/rules/no-async-promise-executor.js +++ b/lib/rules/no-async-promise-executor.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow using an async function as a Promise executor", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-async-promise-executor" }, diff --git a/lib/rules/no-await-in-loop.js b/lib/rules/no-await-in-loop.js index 9ca89866a6e..38af8b56c1a 100644 --- a/lib/rules/no-await-in-loop.js +++ b/lib/rules/no-await-in-loop.js @@ -59,7 +59,6 @@ module.exports = { docs: { description: "disallow `await` inside of loops", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/no-await-in-loop" }, diff --git a/lib/rules/no-bitwise.js b/lib/rules/no-bitwise.js index 1c9d8768155..10bf24a2997 100644 --- a/lib/rules/no-bitwise.js +++ b/lib/rules/no-bitwise.js @@ -26,7 +26,6 @@ module.exports = { docs: { description: "disallow bitwise operators", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-bitwise" }, diff --git a/lib/rules/no-buffer-constructor.js b/lib/rules/no-buffer-constructor.js index 152dda0ceae..832287607f3 100644 --- a/lib/rules/no-buffer-constructor.js +++ b/lib/rules/no-buffer-constructor.js @@ -18,7 +18,6 @@ module.exports = { docs: { description: "disallow use of the `Buffer()` constructor", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/no-buffer-constructor" }, diff --git a/lib/rules/no-caller.js b/lib/rules/no-caller.js index 5fe1bd44985..dbb527906f2 100644 --- a/lib/rules/no-caller.js +++ b/lib/rules/no-caller.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow the use of `arguments.caller` or `arguments.callee`", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-caller" }, diff --git a/lib/rules/no-case-declarations.js b/lib/rules/no-case-declarations.js index 1d54e221625..a132f0370fa 100644 --- a/lib/rules/no-case-declarations.js +++ b/lib/rules/no-case-declarations.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow lexical declarations in case clauses", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-case-declarations" }, diff --git a/lib/rules/no-catch-shadow.js b/lib/rules/no-catch-shadow.js index 4917af84e43..0cbeedf90b6 100644 --- a/lib/rules/no-catch-shadow.js +++ b/lib/rules/no-catch-shadow.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "disallow `catch` clause parameters from shadowing variables in the outer scope", - category: "Variables", recommended: false, url: "https://eslint.org/docs/rules/no-catch-shadow" }, diff --git a/lib/rules/no-class-assign.js b/lib/rules/no-class-assign.js index 887058ba0f9..839ad03e292 100644 --- a/lib/rules/no-class-assign.js +++ b/lib/rules/no-class-assign.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow reassigning class members", - category: "ECMAScript 6", recommended: true, url: "https://eslint.org/docs/rules/no-class-assign" }, diff --git a/lib/rules/no-compare-neg-zero.js b/lib/rules/no-compare-neg-zero.js index 0c6865ad59e..e8fdaa0cc69 100644 --- a/lib/rules/no-compare-neg-zero.js +++ b/lib/rules/no-compare-neg-zero.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow comparing against -0", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-compare-neg-zero" }, diff --git a/lib/rules/no-cond-assign.js b/lib/rules/no-cond-assign.js index 3843a7ac2e3..42f75af7d0c 100644 --- a/lib/rules/no-cond-assign.js +++ b/lib/rules/no-cond-assign.js @@ -34,7 +34,6 @@ module.exports = { docs: { description: "disallow assignment operators in conditional expressions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-cond-assign" }, diff --git a/lib/rules/no-confusing-arrow.js b/lib/rules/no-confusing-arrow.js index 9009b64fa15..fa87f4012e8 100644 --- a/lib/rules/no-confusing-arrow.js +++ b/lib/rules/no-confusing-arrow.js @@ -31,7 +31,6 @@ module.exports = { docs: { description: "disallow arrow functions where they could be confused with comparisons", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/no-confusing-arrow" }, diff --git a/lib/rules/no-console.js b/lib/rules/no-console.js index 56dbbc3a9fd..a5937cbddb2 100644 --- a/lib/rules/no-console.js +++ b/lib/rules/no-console.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow the use of `console`", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/no-console" }, diff --git a/lib/rules/no-const-assign.js b/lib/rules/no-const-assign.js index e4ae891705f..6ca1b6107a8 100644 --- a/lib/rules/no-const-assign.js +++ b/lib/rules/no-const-assign.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow reassigning `const` variables", - category: "ECMAScript 6", recommended: true, url: "https://eslint.org/docs/rules/no-const-assign" }, diff --git a/lib/rules/no-constant-condition.js b/lib/rules/no-constant-condition.js index 3c2d68cbf6c..7a7030a9a46 100644 --- a/lib/rules/no-constant-condition.js +++ b/lib/rules/no-constant-condition.js @@ -19,7 +19,6 @@ module.exports = { docs: { description: "disallow constant expressions in conditions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-constant-condition" }, diff --git a/lib/rules/no-constructor-return.js b/lib/rules/no-constructor-return.js index 4757770b7cc..b4b5baf743c 100644 --- a/lib/rules/no-constructor-return.js +++ b/lib/rules/no-constructor-return.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow returning value from constructor", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-constructor-return" }, diff --git a/lib/rules/no-continue.js b/lib/rules/no-continue.js index 96718d17a3d..e72e862df59 100644 --- a/lib/rules/no-continue.js +++ b/lib/rules/no-continue.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow `continue` statements", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-continue" }, diff --git a/lib/rules/no-control-regex.js b/lib/rules/no-control-regex.js index 6feeb6419d5..908d61ae449 100644 --- a/lib/rules/no-control-regex.js +++ b/lib/rules/no-control-regex.js @@ -52,7 +52,6 @@ module.exports = { docs: { description: "disallow control characters in regular expressions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-control-regex" }, diff --git a/lib/rules/no-debugger.js b/lib/rules/no-debugger.js index 95a28a86215..46dd57639cf 100644 --- a/lib/rules/no-debugger.js +++ b/lib/rules/no-debugger.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow the use of `debugger`", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-debugger" }, diff --git a/lib/rules/no-delete-var.js b/lib/rules/no-delete-var.js index aeab951d75f..1438ebc33bd 100644 --- a/lib/rules/no-delete-var.js +++ b/lib/rules/no-delete-var.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow deleting variables", - category: "Variables", recommended: true, url: "https://eslint.org/docs/rules/no-delete-var" }, diff --git a/lib/rules/no-div-regex.js b/lib/rules/no-div-regex.js index 0ccabdcc698..40388c366e2 100644 --- a/lib/rules/no-div-regex.js +++ b/lib/rules/no-div-regex.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow division operators explicitly at the beginning of regular expressions", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-div-regex" }, diff --git a/lib/rules/no-dupe-args.js b/lib/rules/no-dupe-args.js index 817277f522e..0880b9c8124 100644 --- a/lib/rules/no-dupe-args.js +++ b/lib/rules/no-dupe-args.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow duplicate arguments in `function` definitions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-dupe-args" }, diff --git a/lib/rules/no-dupe-class-members.js b/lib/rules/no-dupe-class-members.js index 31d062370ad..f74865b82a8 100644 --- a/lib/rules/no-dupe-class-members.js +++ b/lib/rules/no-dupe-class-members.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow duplicate class members", - category: "ECMAScript 6", recommended: true, url: "https://eslint.org/docs/rules/no-dupe-class-members" }, diff --git a/lib/rules/no-dupe-else-if.js b/lib/rules/no-dupe-else-if.js index cbeb437da1e..0d8b17cc0ab 100644 --- a/lib/rules/no-dupe-else-if.js +++ b/lib/rules/no-dupe-else-if.js @@ -52,7 +52,6 @@ module.exports = { docs: { description: "disallow duplicate conditions in if-else-if chains", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-dupe-else-if" }, diff --git a/lib/rules/no-dupe-keys.js b/lib/rules/no-dupe-keys.js index cfddde47666..ecec022185f 100644 --- a/lib/rules/no-dupe-keys.js +++ b/lib/rules/no-dupe-keys.js @@ -88,7 +88,6 @@ module.exports = { docs: { description: "disallow duplicate keys in object literals", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-dupe-keys" }, diff --git a/lib/rules/no-duplicate-case.js b/lib/rules/no-duplicate-case.js index e2d9665e7f5..4669dcee1ba 100644 --- a/lib/rules/no-duplicate-case.js +++ b/lib/rules/no-duplicate-case.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "disallow duplicate case labels", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-duplicate-case" }, diff --git a/lib/rules/no-duplicate-imports.js b/lib/rules/no-duplicate-imports.js index cc3da1d5a68..2663698dc96 100644 --- a/lib/rules/no-duplicate-imports.js +++ b/lib/rules/no-duplicate-imports.js @@ -233,7 +233,6 @@ module.exports = { docs: { description: "disallow duplicate module imports", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/no-duplicate-imports" }, diff --git a/lib/rules/no-else-return.js b/lib/rules/no-else-return.js index 84409fac87c..4c981ae1592 100644 --- a/lib/rules/no-else-return.js +++ b/lib/rules/no-else-return.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "disallow `else` blocks after `return` statements in `if` statements", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-else-return" }, diff --git a/lib/rules/no-empty-character-class.js b/lib/rules/no-empty-character-class.js index 301f03bf3d1..85e8ef76791 100644 --- a/lib/rules/no-empty-character-class.js +++ b/lib/rules/no-empty-character-class.js @@ -30,7 +30,6 @@ module.exports = { docs: { description: "disallow empty character classes in regular expressions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-empty-character-class" }, diff --git a/lib/rules/no-empty-function.js b/lib/rules/no-empty-function.js index c512f8cd5f4..8b1073a59d8 100644 --- a/lib/rules/no-empty-function.js +++ b/lib/rules/no-empty-function.js @@ -95,7 +95,6 @@ module.exports = { docs: { description: "disallow empty functions", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-empty-function" }, diff --git a/lib/rules/no-empty-pattern.js b/lib/rules/no-empty-pattern.js index 9f34bfde92e..99ea3a7905b 100644 --- a/lib/rules/no-empty-pattern.js +++ b/lib/rules/no-empty-pattern.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow empty destructuring patterns", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-empty-pattern" }, diff --git a/lib/rules/no-empty.js b/lib/rules/no-empty.js index 45bf03c13ae..4ed3c5c5458 100644 --- a/lib/rules/no-empty.js +++ b/lib/rules/no-empty.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "disallow empty block statements", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-empty" }, diff --git a/lib/rules/no-eq-null.js b/lib/rules/no-eq-null.js index b8dead96d25..dae922840b8 100644 --- a/lib/rules/no-eq-null.js +++ b/lib/rules/no-eq-null.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow `null` comparisons without type-checking operators", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-eq-null" }, diff --git a/lib/rules/no-eval.js b/lib/rules/no-eval.js index 4360ac4110e..97481528357 100644 --- a/lib/rules/no-eval.js +++ b/lib/rules/no-eval.js @@ -43,7 +43,6 @@ module.exports = { docs: { description: "disallow the use of `eval()`", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-eval" }, diff --git a/lib/rules/no-ex-assign.js b/lib/rules/no-ex-assign.js index 1163920361d..cd56c94af75 100644 --- a/lib/rules/no-ex-assign.js +++ b/lib/rules/no-ex-assign.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow reassigning exceptions in `catch` clauses", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-ex-assign" }, diff --git a/lib/rules/no-extend-native.js b/lib/rules/no-extend-native.js index 2a804b56398..4d5accbae63 100644 --- a/lib/rules/no-extend-native.js +++ b/lib/rules/no-extend-native.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "disallow extending native types", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-extend-native" }, diff --git a/lib/rules/no-extra-bind.js b/lib/rules/no-extra-bind.js index 2db440dc1ea..6fd3be1d601 100644 --- a/lib/rules/no-extra-bind.js +++ b/lib/rules/no-extra-bind.js @@ -26,7 +26,6 @@ module.exports = { docs: { description: "disallow unnecessary calls to `.bind()`", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-extra-bind" }, diff --git a/lib/rules/no-extra-boolean-cast.js b/lib/rules/no-extra-boolean-cast.js index e68f2941eb0..cb061dac5d5 100644 --- a/lib/rules/no-extra-boolean-cast.js +++ b/lib/rules/no-extra-boolean-cast.js @@ -24,7 +24,6 @@ module.exports = { docs: { description: "disallow unnecessary boolean casts", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-extra-boolean-cast" }, diff --git a/lib/rules/no-extra-label.js b/lib/rules/no-extra-label.js index 81406e76095..bbb2413b2c7 100644 --- a/lib/rules/no-extra-label.js +++ b/lib/rules/no-extra-label.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow unnecessary labels", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-extra-label" }, diff --git a/lib/rules/no-extra-parens.js b/lib/rules/no-extra-parens.js index d063350461d..0756d2fb2c7 100644 --- a/lib/rules/no-extra-parens.js +++ b/lib/rules/no-extra-parens.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow unnecessary parentheses", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/no-extra-parens" }, diff --git a/lib/rules/no-extra-semi.js b/lib/rules/no-extra-semi.js index 42c24427aa5..952869c3ea7 100644 --- a/lib/rules/no-extra-semi.js +++ b/lib/rules/no-extra-semi.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "disallow unnecessary semicolons", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-extra-semi" }, diff --git a/lib/rules/no-fallthrough.js b/lib/rules/no-fallthrough.js index 3b949acd1da..bf2c82514bb 100644 --- a/lib/rules/no-fallthrough.js +++ b/lib/rules/no-fallthrough.js @@ -64,7 +64,6 @@ module.exports = { docs: { description: "disallow fallthrough of `case` statements", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-fallthrough" }, diff --git a/lib/rules/no-floating-decimal.js b/lib/rules/no-floating-decimal.js index b1d883212e2..92ac2326b9f 100644 --- a/lib/rules/no-floating-decimal.js +++ b/lib/rules/no-floating-decimal.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow leading or trailing decimal points in numeric literals", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-floating-decimal" }, diff --git a/lib/rules/no-func-assign.js b/lib/rules/no-func-assign.js index 33d0ad9ecd1..aa04f337ae0 100644 --- a/lib/rules/no-func-assign.js +++ b/lib/rules/no-func-assign.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow reassigning `function` declarations", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-func-assign" }, diff --git a/lib/rules/no-global-assign.js b/lib/rules/no-global-assign.js index ea854c4aa8c..85aac7cdc0f 100644 --- a/lib/rules/no-global-assign.js +++ b/lib/rules/no-global-assign.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow assignments to native objects or read-only global variables", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-global-assign" }, diff --git a/lib/rules/no-implicit-coercion.js b/lib/rules/no-implicit-coercion.js index 993b8d1f1c8..1d11e10d597 100644 --- a/lib/rules/no-implicit-coercion.js +++ b/lib/rules/no-implicit-coercion.js @@ -173,7 +173,6 @@ module.exports = { docs: { description: "disallow shorthand type conversions", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-implicit-coercion" }, diff --git a/lib/rules/no-implicit-globals.js b/lib/rules/no-implicit-globals.js index d4bfa3af82f..8740cd80531 100644 --- a/lib/rules/no-implicit-globals.js +++ b/lib/rules/no-implicit-globals.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow declarations in the global scope", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-implicit-globals" }, diff --git a/lib/rules/no-implied-eval.js b/lib/rules/no-implied-eval.js index b8120a64887..2432e68b612 100644 --- a/lib/rules/no-implied-eval.js +++ b/lib/rules/no-implied-eval.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "disallow the use of `eval()`-like methods", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-implied-eval" }, diff --git a/lib/rules/no-import-assign.js b/lib/rules/no-import-assign.js index 41060d8ac9e..fbe63d0539f 100644 --- a/lib/rules/no-import-assign.js +++ b/lib/rules/no-import-assign.js @@ -180,7 +180,6 @@ module.exports = { docs: { description: "disallow assigning to imported bindings", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-import-assign" }, diff --git a/lib/rules/no-inline-comments.js b/lib/rules/no-inline-comments.js index dec278615e2..8a955a6130e 100644 --- a/lib/rules/no-inline-comments.js +++ b/lib/rules/no-inline-comments.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow inline comments after code", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-inline-comments" }, diff --git a/lib/rules/no-inner-declarations.js b/lib/rules/no-inner-declarations.js index 0768bc61149..9bbe24d7c82 100644 --- a/lib/rules/no-inner-declarations.js +++ b/lib/rules/no-inner-declarations.js @@ -24,7 +24,6 @@ module.exports = { docs: { description: "disallow variable or `function` declarations in nested blocks", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-inner-declarations" }, diff --git a/lib/rules/no-invalid-regexp.js b/lib/rules/no-invalid-regexp.js index e3c8682e65b..ee199328966 100644 --- a/lib/rules/no-invalid-regexp.js +++ b/lib/rules/no-invalid-regexp.js @@ -23,7 +23,6 @@ module.exports = { docs: { description: "disallow invalid regular expression strings in `RegExp` constructors", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-invalid-regexp" }, diff --git a/lib/rules/no-invalid-this.js b/lib/rules/no-invalid-this.js index 5bb5ea1e0b9..77558b90dcc 100644 --- a/lib/rules/no-invalid-this.js +++ b/lib/rules/no-invalid-this.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow `this` keywords outside of classes or class-like objects", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-invalid-this" }, diff --git a/lib/rules/no-irregular-whitespace.js b/lib/rules/no-irregular-whitespace.js index 15711c6157a..c1609715394 100644 --- a/lib/rules/no-irregular-whitespace.js +++ b/lib/rules/no-irregular-whitespace.js @@ -31,7 +31,6 @@ module.exports = { docs: { description: "disallow irregular whitespace", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-irregular-whitespace" }, diff --git a/lib/rules/no-iterator.js b/lib/rules/no-iterator.js index 9ba1e7aefdb..4117f6211c7 100644 --- a/lib/rules/no-iterator.js +++ b/lib/rules/no-iterator.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow the use of the `__iterator__` property", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-iterator" }, diff --git a/lib/rules/no-label-var.js b/lib/rules/no-label-var.js index 570db03285c..4532527c6e8 100644 --- a/lib/rules/no-label-var.js +++ b/lib/rules/no-label-var.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow labels that share a name with a variable", - category: "Variables", recommended: false, url: "https://eslint.org/docs/rules/no-label-var" }, diff --git a/lib/rules/no-labels.js b/lib/rules/no-labels.js index 85760d80dbe..5dd15be092e 100644 --- a/lib/rules/no-labels.js +++ b/lib/rules/no-labels.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "disallow labeled statements", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-labels" }, diff --git a/lib/rules/no-lone-blocks.js b/lib/rules/no-lone-blocks.js index 290784b82ea..5f74cd83c8c 100644 --- a/lib/rules/no-lone-blocks.js +++ b/lib/rules/no-lone-blocks.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow unnecessary nested blocks", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-lone-blocks" }, diff --git a/lib/rules/no-lonely-if.js b/lib/rules/no-lonely-if.js index 6552adc5752..e44f000141b 100644 --- a/lib/rules/no-lonely-if.js +++ b/lib/rules/no-lonely-if.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow `if` statements as the only statement in `else` blocks", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-lonely-if" }, diff --git a/lib/rules/no-loop-func.js b/lib/rules/no-loop-func.js index 40d81900f8a..d1a7868072a 100644 --- a/lib/rules/no-loop-func.js +++ b/lib/rules/no-loop-func.js @@ -154,7 +154,6 @@ module.exports = { docs: { description: "disallow function declarations that contain unsafe references inside loop statements", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-loop-func" }, diff --git a/lib/rules/no-loss-of-precision.js b/lib/rules/no-loss-of-precision.js index 022bba2525a..417616dd231 100644 --- a/lib/rules/no-loss-of-precision.js +++ b/lib/rules/no-loss-of-precision.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow literal numbers that lose precision", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-loss-of-precision" }, diff --git a/lib/rules/no-magic-numbers.js b/lib/rules/no-magic-numbers.js index 510b3f9b261..a2c678e7100 100644 --- a/lib/rules/no-magic-numbers.js +++ b/lib/rules/no-magic-numbers.js @@ -32,7 +32,6 @@ module.exports = { docs: { description: "disallow magic numbers", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-magic-numbers" }, diff --git a/lib/rules/no-misleading-character-class.js b/lib/rules/no-misleading-character-class.js index 3d004615c3f..70e31e604f4 100644 --- a/lib/rules/no-misleading-character-class.js +++ b/lib/rules/no-misleading-character-class.js @@ -104,7 +104,6 @@ module.exports = { docs: { description: "disallow characters which are made with multiple code points in character class syntax", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-misleading-character-class" }, diff --git a/lib/rules/no-mixed-operators.js b/lib/rules/no-mixed-operators.js index 2718e2530c6..ed37a90b1c6 100644 --- a/lib/rules/no-mixed-operators.js +++ b/lib/rules/no-mixed-operators.js @@ -88,7 +88,6 @@ module.exports = { docs: { description: "disallow mixed binary operators", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-mixed-operators" }, diff --git a/lib/rules/no-mixed-requires.js b/lib/rules/no-mixed-requires.js index a02de9104bd..de981d7b33a 100644 --- a/lib/rules/no-mixed-requires.js +++ b/lib/rules/no-mixed-requires.js @@ -19,7 +19,6 @@ module.exports = { docs: { description: "disallow `require` calls to be mixed with regular variable declarations", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/no-mixed-requires" }, diff --git a/lib/rules/no-mixed-spaces-and-tabs.js b/lib/rules/no-mixed-spaces-and-tabs.js index 287cbda03da..ac73cddda3b 100644 --- a/lib/rules/no-mixed-spaces-and-tabs.js +++ b/lib/rules/no-mixed-spaces-and-tabs.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow mixed spaces and tabs for indentation", - category: "Stylistic Issues", recommended: true, url: "https://eslint.org/docs/rules/no-mixed-spaces-and-tabs" }, diff --git a/lib/rules/no-multi-assign.js b/lib/rules/no-multi-assign.js index d517a6beb95..8d7bd32c7dd 100644 --- a/lib/rules/no-multi-assign.js +++ b/lib/rules/no-multi-assign.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow use of chained assignment expressions", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-multi-assign" }, diff --git a/lib/rules/no-multi-spaces.js b/lib/rules/no-multi-spaces.js index d43ed736337..0134dd279b5 100644 --- a/lib/rules/no-multi-spaces.js +++ b/lib/rules/no-multi-spaces.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow multiple spaces", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-multi-spaces" }, diff --git a/lib/rules/no-multi-str.js b/lib/rules/no-multi-str.js index 7cf1ae36794..848f8d405a8 100644 --- a/lib/rules/no-multi-str.js +++ b/lib/rules/no-multi-str.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow multiline strings", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-multi-str" }, diff --git a/lib/rules/no-multiple-empty-lines.js b/lib/rules/no-multiple-empty-lines.js index 9cccef3088a..33ac76f6037 100644 --- a/lib/rules/no-multiple-empty-lines.js +++ b/lib/rules/no-multiple-empty-lines.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow multiple empty lines", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-multiple-empty-lines" }, diff --git a/lib/rules/no-native-reassign.js b/lib/rules/no-native-reassign.js index 833e3b7ce40..80ba0948cbc 100644 --- a/lib/rules/no-native-reassign.js +++ b/lib/rules/no-native-reassign.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow assignments to native objects or read-only global variables", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-native-reassign" }, diff --git a/lib/rules/no-negated-condition.js b/lib/rules/no-negated-condition.js index 8a9eba881df..b5cbadca50f 100644 --- a/lib/rules/no-negated-condition.js +++ b/lib/rules/no-negated-condition.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow negated conditions", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-negated-condition" }, diff --git a/lib/rules/no-negated-in-lhs.js b/lib/rules/no-negated-in-lhs.js index 1229cedd119..0f9c84be6c9 100644 --- a/lib/rules/no-negated-in-lhs.js +++ b/lib/rules/no-negated-in-lhs.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow negating the left operand in `in` expressions", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/no-negated-in-lhs" }, diff --git a/lib/rules/no-nested-ternary.js b/lib/rules/no-nested-ternary.js index 383bb238887..2d3359d38f1 100644 --- a/lib/rules/no-nested-ternary.js +++ b/lib/rules/no-nested-ternary.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow nested ternary expressions", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-nested-ternary" }, diff --git a/lib/rules/no-new-func.js b/lib/rules/no-new-func.js index 9af4e31cabf..eebc68c80c2 100644 --- a/lib/rules/no-new-func.js +++ b/lib/rules/no-new-func.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow `new` operators with the `Function` object", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-new-func" }, diff --git a/lib/rules/no-new-object.js b/lib/rules/no-new-object.js index e9f915db5ea..17dfd344476 100644 --- a/lib/rules/no-new-object.js +++ b/lib/rules/no-new-object.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow `Object` constructors", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-new-object" }, diff --git a/lib/rules/no-new-require.js b/lib/rules/no-new-require.js index 063f783e70c..12745c02b71 100644 --- a/lib/rules/no-new-require.js +++ b/lib/rules/no-new-require.js @@ -19,7 +19,6 @@ module.exports = { docs: { description: "disallow `new` operators with calls to `require`", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/no-new-require" }, diff --git a/lib/rules/no-new-symbol.js b/lib/rules/no-new-symbol.js index aeb509c0d9c..391527df90c 100644 --- a/lib/rules/no-new-symbol.js +++ b/lib/rules/no-new-symbol.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow `new` operators with the `Symbol` object", - category: "ECMAScript 6", recommended: true, url: "https://eslint.org/docs/rules/no-new-symbol" }, diff --git a/lib/rules/no-new-wrappers.js b/lib/rules/no-new-wrappers.js index d276c48d203..b697d8d7951 100644 --- a/lib/rules/no-new-wrappers.js +++ b/lib/rules/no-new-wrappers.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow `new` operators with the `String`, `Number`, and `Boolean` objects", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-new-wrappers" }, diff --git a/lib/rules/no-new.js b/lib/rules/no-new.js index aa8a4e26876..1b37f077d5c 100644 --- a/lib/rules/no-new.js +++ b/lib/rules/no-new.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow `new` operators outside of assignments or comparisons", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-new" }, diff --git a/lib/rules/no-nonoctal-decimal-escape.js b/lib/rules/no-nonoctal-decimal-escape.js index deaafac550f..da61f61d02f 100644 --- a/lib/rules/no-nonoctal-decimal-escape.js +++ b/lib/rules/no-nonoctal-decimal-escape.js @@ -30,7 +30,6 @@ module.exports = { docs: { description: "disallow `\\8` and `\\9` escape sequences in string literals", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-nonoctal-decimal-escape" }, diff --git a/lib/rules/no-obj-calls.js b/lib/rules/no-obj-calls.js index 6eb200c9b87..d62c1f0b4be 100644 --- a/lib/rules/no-obj-calls.js +++ b/lib/rules/no-obj-calls.js @@ -43,7 +43,6 @@ module.exports = { docs: { description: "disallow calling global object properties as functions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-obj-calls" }, diff --git a/lib/rules/no-octal-escape.js b/lib/rules/no-octal-escape.js index 5b4c7b2ebb4..4513a83861d 100644 --- a/lib/rules/no-octal-escape.js +++ b/lib/rules/no-octal-escape.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow octal escape sequences in string literals", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-octal-escape" }, diff --git a/lib/rules/no-octal.js b/lib/rules/no-octal.js index e9940befafa..5ee6895f623 100644 --- a/lib/rules/no-octal.js +++ b/lib/rules/no-octal.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow octal literals", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-octal" }, diff --git a/lib/rules/no-param-reassign.js b/lib/rules/no-param-reassign.js index 6874af44f38..b758b9d97fc 100644 --- a/lib/rules/no-param-reassign.js +++ b/lib/rules/no-param-reassign.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow reassigning `function` parameters", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-param-reassign" }, diff --git a/lib/rules/no-path-concat.js b/lib/rules/no-path-concat.js index fc1f894f878..0671e511e86 100644 --- a/lib/rules/no-path-concat.js +++ b/lib/rules/no-path-concat.js @@ -18,7 +18,6 @@ module.exports = { docs: { description: "disallow string concatenation with `__dirname` and `__filename`", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/no-path-concat" }, diff --git a/lib/rules/no-plusplus.js b/lib/rules/no-plusplus.js index 84d6c3e1f91..d7b6c730562 100644 --- a/lib/rules/no-plusplus.js +++ b/lib/rules/no-plusplus.js @@ -51,7 +51,6 @@ module.exports = { docs: { description: "disallow the unary operators `++` and `--`", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-plusplus" }, diff --git a/lib/rules/no-process-env.js b/lib/rules/no-process-env.js index 49d1734906e..3ba66d6f5a2 100644 --- a/lib/rules/no-process-env.js +++ b/lib/rules/no-process-env.js @@ -18,7 +18,6 @@ module.exports = { docs: { description: "disallow the use of `process.env`", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/no-process-env" }, diff --git a/lib/rules/no-process-exit.js b/lib/rules/no-process-exit.js index 77c9cfd7cbd..de6de4fabc4 100644 --- a/lib/rules/no-process-exit.js +++ b/lib/rules/no-process-exit.js @@ -18,7 +18,6 @@ module.exports = { docs: { description: "disallow the use of `process.exit()`", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/no-process-exit" }, diff --git a/lib/rules/no-promise-executor-return.js b/lib/rules/no-promise-executor-return.js index 32ee6e15124..42652416f8f 100644 --- a/lib/rules/no-promise-executor-return.js +++ b/lib/rules/no-promise-executor-return.js @@ -69,7 +69,6 @@ module.exports = { docs: { description: "disallow returning values from Promise executor functions", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/no-promise-executor-return" }, diff --git a/lib/rules/no-proto.js b/lib/rules/no-proto.js index 82ce02fa4e3..0c2490f7b42 100644 --- a/lib/rules/no-proto.js +++ b/lib/rules/no-proto.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow the use of the `__proto__` property", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-proto" }, diff --git a/lib/rules/no-prototype-builtins.js b/lib/rules/no-prototype-builtins.js index c5e4d49259b..1f837b96040 100644 --- a/lib/rules/no-prototype-builtins.js +++ b/lib/rules/no-prototype-builtins.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "disallow calling some `Object.prototype` methods directly on objects", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-prototype-builtins" }, diff --git a/lib/rules/no-redeclare.js b/lib/rules/no-redeclare.js index 6ddb21c9e15..afbe6170cb7 100644 --- a/lib/rules/no-redeclare.js +++ b/lib/rules/no-redeclare.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow variable redeclaration", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-redeclare" }, diff --git a/lib/rules/no-regex-spaces.js b/lib/rules/no-regex-spaces.js index e6d4c9efba7..1d6b121ba80 100644 --- a/lib/rules/no-regex-spaces.js +++ b/lib/rules/no-regex-spaces.js @@ -39,7 +39,6 @@ module.exports = { docs: { description: "disallow multiple spaces in regular expressions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-regex-spaces" }, diff --git a/lib/rules/no-restricted-exports.js b/lib/rules/no-restricted-exports.js index f0df0ffaedb..f568fdc6850 100644 --- a/lib/rules/no-restricted-exports.js +++ b/lib/rules/no-restricted-exports.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow specified names in exports", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/no-restricted-exports" }, diff --git a/lib/rules/no-restricted-globals.js b/lib/rules/no-restricted-globals.js index 9193c57cd60..efbcd755b1f 100644 --- a/lib/rules/no-restricted-globals.js +++ b/lib/rules/no-restricted-globals.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow specified global variables", - category: "Variables", recommended: false, url: "https://eslint.org/docs/rules/no-restricted-globals" }, diff --git a/lib/rules/no-restricted-imports.js b/lib/rules/no-restricted-imports.js index aa9a9a87385..eda63407ff2 100644 --- a/lib/rules/no-restricted-imports.js +++ b/lib/rules/no-restricted-imports.js @@ -79,7 +79,6 @@ module.exports = { docs: { description: "disallow specified modules when loaded by `import`", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/no-restricted-imports" }, diff --git a/lib/rules/no-restricted-modules.js b/lib/rules/no-restricted-modules.js index b4d798daea8..ebf88dcb6ce 100644 --- a/lib/rules/no-restricted-modules.js +++ b/lib/rules/no-restricted-modules.js @@ -48,7 +48,6 @@ module.exports = { docs: { description: "disallow specified modules when loaded by `require`", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/no-restricted-modules" }, diff --git a/lib/rules/no-restricted-properties.js b/lib/rules/no-restricted-properties.js index 00b09a30bc0..3671d88eb60 100644 --- a/lib/rules/no-restricted-properties.js +++ b/lib/rules/no-restricted-properties.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow certain properties on certain objects", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-restricted-properties" }, diff --git a/lib/rules/no-restricted-syntax.js b/lib/rules/no-restricted-syntax.js index e82dd3d56e0..0ff6b91bc69 100644 --- a/lib/rules/no-restricted-syntax.js +++ b/lib/rules/no-restricted-syntax.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow specified syntax", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-restricted-syntax" }, diff --git a/lib/rules/no-return-assign.js b/lib/rules/no-return-assign.js index 4b57d42eb99..ecb789ea269 100644 --- a/lib/rules/no-return-assign.js +++ b/lib/rules/no-return-assign.js @@ -26,7 +26,6 @@ module.exports = { docs: { description: "disallow assignment operators in `return` statements", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-return-assign" }, diff --git a/lib/rules/no-return-await.js b/lib/rules/no-return-await.js index d1d89826856..7ec808f50a3 100644 --- a/lib/rules/no-return-await.js +++ b/lib/rules/no-return-await.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow unnecessary `return await`", - category: "Best Practices", recommended: false, diff --git a/lib/rules/no-script-url.js b/lib/rules/no-script-url.js index 9544730fdcc..12451ad9a9e 100644 --- a/lib/rules/no-script-url.js +++ b/lib/rules/no-script-url.js @@ -18,7 +18,6 @@ module.exports = { docs: { description: "disallow `javascript:` urls", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-script-url" }, diff --git a/lib/rules/no-self-assign.js b/lib/rules/no-self-assign.js index 705be324cf0..813771e700a 100644 --- a/lib/rules/no-self-assign.js +++ b/lib/rules/no-self-assign.js @@ -130,7 +130,6 @@ module.exports = { docs: { description: "disallow assignments where both sides are exactly the same", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-self-assign" }, diff --git a/lib/rules/no-self-compare.js b/lib/rules/no-self-compare.js index 79b6ac7ea0f..ee77ff08b20 100644 --- a/lib/rules/no-self-compare.js +++ b/lib/rules/no-self-compare.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow comparisons where both sides are exactly the same", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-self-compare" }, diff --git a/lib/rules/no-sequences.js b/lib/rules/no-sequences.js index fe516975fbc..b8941256e6f 100644 --- a/lib/rules/no-sequences.js +++ b/lib/rules/no-sequences.js @@ -29,7 +29,6 @@ module.exports = { docs: { description: "disallow comma operators", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-sequences" }, diff --git a/lib/rules/no-setter-return.js b/lib/rules/no-setter-return.js index 90c2095ac20..67114ade821 100644 --- a/lib/rules/no-setter-return.js +++ b/lib/rules/no-setter-return.js @@ -142,7 +142,6 @@ module.exports = { docs: { description: "disallow returning values from setters", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-setter-return" }, diff --git a/lib/rules/no-shadow-restricted-names.js b/lib/rules/no-shadow-restricted-names.js index 9647e9a1bcd..7d4174a104e 100644 --- a/lib/rules/no-shadow-restricted-names.js +++ b/lib/rules/no-shadow-restricted-names.js @@ -27,7 +27,6 @@ module.exports = { docs: { description: "disallow identifiers from shadowing restricted names", - category: "Variables", recommended: true, url: "https://eslint.org/docs/rules/no-shadow-restricted-names" }, diff --git a/lib/rules/no-shadow.js b/lib/rules/no-shadow.js index 7a8c24e1894..4ec357620b6 100644 --- a/lib/rules/no-shadow.js +++ b/lib/rules/no-shadow.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow variable declarations from shadowing variables declared in the outer scope", - category: "Variables", recommended: false, url: "https://eslint.org/docs/rules/no-shadow" }, diff --git a/lib/rules/no-spaced-func.js b/lib/rules/no-spaced-func.js index 961bc681f7e..8f51d5446d8 100644 --- a/lib/rules/no-spaced-func.js +++ b/lib/rules/no-spaced-func.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "disallow spacing between function identifiers and their applications (deprecated)", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-spaced-func" }, diff --git a/lib/rules/no-sparse-arrays.js b/lib/rules/no-sparse-arrays.js index e8407c3faed..56ce5dcc871 100644 --- a/lib/rules/no-sparse-arrays.js +++ b/lib/rules/no-sparse-arrays.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow sparse arrays", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-sparse-arrays" }, diff --git a/lib/rules/no-sync.js b/lib/rules/no-sync.js index c114a662a5b..a67eb8acf49 100644 --- a/lib/rules/no-sync.js +++ b/lib/rules/no-sync.js @@ -19,7 +19,6 @@ module.exports = { docs: { description: "disallow synchronous methods", - category: "Node.js and CommonJS", recommended: false, url: "https://eslint.org/docs/rules/no-sync" }, diff --git a/lib/rules/no-tabs.js b/lib/rules/no-tabs.js index ca7be261653..1f3921a9a70 100644 --- a/lib/rules/no-tabs.js +++ b/lib/rules/no-tabs.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "disallow all tabs", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-tabs" }, diff --git a/lib/rules/no-template-curly-in-string.js b/lib/rules/no-template-curly-in-string.js index 539cd5be5ff..e71480fc013 100644 --- a/lib/rules/no-template-curly-in-string.js +++ b/lib/rules/no-template-curly-in-string.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow template literal placeholder syntax in regular strings", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/no-template-curly-in-string" }, diff --git a/lib/rules/no-ternary.js b/lib/rules/no-ternary.js index b3ced860566..8b2e10a34a4 100644 --- a/lib/rules/no-ternary.js +++ b/lib/rules/no-ternary.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow ternary operators", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-ternary" }, diff --git a/lib/rules/no-this-before-super.js b/lib/rules/no-this-before-super.js index 5bfba66fc65..9cc85ebbe21 100644 --- a/lib/rules/no-this-before-super.js +++ b/lib/rules/no-this-before-super.js @@ -40,7 +40,6 @@ module.exports = { docs: { description: "disallow `this`/`super` before calling `super()` in constructors", - category: "ECMAScript 6", recommended: true, url: "https://eslint.org/docs/rules/no-this-before-super" }, diff --git a/lib/rules/no-throw-literal.js b/lib/rules/no-throw-literal.js index 29fb3718b48..311e6d4f079 100644 --- a/lib/rules/no-throw-literal.js +++ b/lib/rules/no-throw-literal.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow throwing literals as exceptions", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-throw-literal" }, diff --git a/lib/rules/no-trailing-spaces.js b/lib/rules/no-trailing-spaces.js index 98ae62c8963..1930098bea0 100644 --- a/lib/rules/no-trailing-spaces.js +++ b/lib/rules/no-trailing-spaces.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "disallow trailing whitespace at the end of lines", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-trailing-spaces" }, diff --git a/lib/rules/no-undef-init.js b/lib/rules/no-undef-init.js index e028fe42832..3252fbb250f 100644 --- a/lib/rules/no-undef-init.js +++ b/lib/rules/no-undef-init.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "disallow initializing variables to `undefined`", - category: "Variables", recommended: false, url: "https://eslint.org/docs/rules/no-undef-init" }, diff --git a/lib/rules/no-undef.js b/lib/rules/no-undef.js index 6b5140819bb..ee611f9c866 100644 --- a/lib/rules/no-undef.js +++ b/lib/rules/no-undef.js @@ -29,7 +29,6 @@ module.exports = { docs: { description: "disallow the use of undeclared variables unless mentioned in `/*global */` comments", - category: "Variables", recommended: true, url: "https://eslint.org/docs/rules/no-undef" }, diff --git a/lib/rules/no-undefined.js b/lib/rules/no-undefined.js index a075d903e21..ad302255420 100644 --- a/lib/rules/no-undefined.js +++ b/lib/rules/no-undefined.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow the use of `undefined` as an identifier", - category: "Variables", recommended: false, url: "https://eslint.org/docs/rules/no-undefined" }, diff --git a/lib/rules/no-underscore-dangle.js b/lib/rules/no-underscore-dangle.js index 6dc0d46f218..916b8c01baa 100644 --- a/lib/rules/no-underscore-dangle.js +++ b/lib/rules/no-underscore-dangle.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow dangling underscores in identifiers", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-underscore-dangle" }, diff --git a/lib/rules/no-unexpected-multiline.js b/lib/rules/no-unexpected-multiline.js index 7af3fe67090..4447959ed9a 100644 --- a/lib/rules/no-unexpected-multiline.js +++ b/lib/rules/no-unexpected-multiline.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "disallow confusing multiline expressions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-unexpected-multiline" }, diff --git a/lib/rules/no-unmodified-loop-condition.js b/lib/rules/no-unmodified-loop-condition.js index 7031a4dd8b8..ba321d2c6ed 100644 --- a/lib/rules/no-unmodified-loop-condition.js +++ b/lib/rules/no-unmodified-loop-condition.js @@ -162,7 +162,6 @@ module.exports = { docs: { description: "disallow unmodified loop conditions", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-unmodified-loop-condition" }, diff --git a/lib/rules/no-unneeded-ternary.js b/lib/rules/no-unneeded-ternary.js index 06c615f3824..e12240d03e0 100644 --- a/lib/rules/no-unneeded-ternary.js +++ b/lib/rules/no-unneeded-ternary.js @@ -29,7 +29,6 @@ module.exports = { docs: { description: "disallow ternary operators when simpler alternatives exist", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-unneeded-ternary" }, diff --git a/lib/rules/no-unreachable-loop.js b/lib/rules/no-unreachable-loop.js index 868a6ff98f8..5cbfac46d31 100644 --- a/lib/rules/no-unreachable-loop.js +++ b/lib/rules/no-unreachable-loop.js @@ -59,7 +59,6 @@ module.exports = { docs: { description: "disallow loops with a body that allows only one iteration", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/no-unreachable-loop" }, diff --git a/lib/rules/no-unreachable.js b/lib/rules/no-unreachable.js index 586be02e3cc..ce17a5966a7 100644 --- a/lib/rules/no-unreachable.js +++ b/lib/rules/no-unreachable.js @@ -111,7 +111,6 @@ module.exports = { docs: { description: "disallow unreachable code after `return`, `throw`, `continue`, and `break` statements", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-unreachable" }, diff --git a/lib/rules/no-unsafe-finally.js b/lib/rules/no-unsafe-finally.js index 11bf06e872a..4bb7f7fec4f 100644 --- a/lib/rules/no-unsafe-finally.js +++ b/lib/rules/no-unsafe-finally.js @@ -24,7 +24,6 @@ module.exports = { docs: { description: "disallow control flow statements in `finally` blocks", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-unsafe-finally" }, diff --git a/lib/rules/no-unsafe-negation.js b/lib/rules/no-unsafe-negation.js index f1d5c9c4e31..c681986941a 100644 --- a/lib/rules/no-unsafe-negation.js +++ b/lib/rules/no-unsafe-negation.js @@ -52,7 +52,6 @@ module.exports = { docs: { description: "disallow negating the left operand of relational operators", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-unsafe-negation" }, diff --git a/lib/rules/no-unsafe-optional-chaining.js b/lib/rules/no-unsafe-optional-chaining.js index e430a3bdea0..cc15c998813 100644 --- a/lib/rules/no-unsafe-optional-chaining.js +++ b/lib/rules/no-unsafe-optional-chaining.js @@ -24,7 +24,6 @@ module.exports = { docs: { description: "disallow use of optional chaining in contexts where the `undefined` value is not allowed", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-unsafe-optional-chaining" }, diff --git a/lib/rules/no-unused-expressions.js b/lib/rules/no-unused-expressions.js index a682ded3ce7..3bb816df778 100644 --- a/lib/rules/no-unused-expressions.js +++ b/lib/rules/no-unused-expressions.js @@ -30,7 +30,6 @@ module.exports = { docs: { description: "disallow unused expressions", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-unused-expressions" }, diff --git a/lib/rules/no-unused-labels.js b/lib/rules/no-unused-labels.js index b33fcb78679..f0b09614e09 100644 --- a/lib/rules/no-unused-labels.js +++ b/lib/rules/no-unused-labels.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow unused labels", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-unused-labels" }, diff --git a/lib/rules/no-unused-vars.js b/lib/rules/no-unused-vars.js index 4733927bc51..847e21baf5f 100644 --- a/lib/rules/no-unused-vars.js +++ b/lib/rules/no-unused-vars.js @@ -33,7 +33,6 @@ module.exports = { docs: { description: "disallow unused variables", - category: "Variables", recommended: true, url: "https://eslint.org/docs/rules/no-unused-vars" }, diff --git a/lib/rules/no-use-before-define.js b/lib/rules/no-use-before-define.js index c7300567ede..fd8864e5fa5 100644 --- a/lib/rules/no-use-before-define.js +++ b/lib/rules/no-use-before-define.js @@ -135,7 +135,6 @@ module.exports = { docs: { description: "disallow the use of variables before they are defined", - category: "Variables", recommended: false, url: "https://eslint.org/docs/rules/no-use-before-define" }, diff --git a/lib/rules/no-useless-backreference.js b/lib/rules/no-useless-backreference.js index cb588d91b35..ae491471282 100644 --- a/lib/rules/no-useless-backreference.js +++ b/lib/rules/no-useless-backreference.js @@ -64,7 +64,6 @@ module.exports = { docs: { description: "disallow useless backreferences in regular expressions", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/no-useless-backreference" }, diff --git a/lib/rules/no-useless-call.js b/lib/rules/no-useless-call.js index b1382a2fa28..89350665fe3 100644 --- a/lib/rules/no-useless-call.js +++ b/lib/rules/no-useless-call.js @@ -55,7 +55,6 @@ module.exports = { docs: { description: "disallow unnecessary calls to `.call()` and `.apply()`", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-useless-call" }, diff --git a/lib/rules/no-useless-catch.js b/lib/rules/no-useless-catch.js index f303c272948..280ba553367 100644 --- a/lib/rules/no-useless-catch.js +++ b/lib/rules/no-useless-catch.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow unnecessary `catch` clauses", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-useless-catch" }, diff --git a/lib/rules/no-useless-computed-key.js b/lib/rules/no-useless-computed-key.js index cedd2ec242a..a8769214f54 100644 --- a/lib/rules/no-useless-computed-key.js +++ b/lib/rules/no-useless-computed-key.js @@ -91,7 +91,6 @@ module.exports = { docs: { description: "disallow unnecessary computed property keys in objects and classes", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/no-useless-computed-key" }, diff --git a/lib/rules/no-useless-concat.js b/lib/rules/no-useless-concat.js index cfc60c8fb51..a0176a7e9db 100644 --- a/lib/rules/no-useless-concat.js +++ b/lib/rules/no-useless-concat.js @@ -70,7 +70,6 @@ module.exports = { docs: { description: "disallow unnecessary concatenation of literals or template literals", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-useless-concat" }, diff --git a/lib/rules/no-useless-constructor.js b/lib/rules/no-useless-constructor.js index baabe7ec80f..13ec6755015 100644 --- a/lib/rules/no-useless-constructor.js +++ b/lib/rules/no-useless-constructor.js @@ -138,7 +138,6 @@ module.exports = { docs: { description: "disallow unnecessary constructors", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/no-useless-constructor" }, diff --git a/lib/rules/no-useless-escape.js b/lib/rules/no-useless-escape.js index 4811b40420a..a780a7a84a6 100644 --- a/lib/rules/no-useless-escape.js +++ b/lib/rules/no-useless-escape.js @@ -84,7 +84,6 @@ module.exports = { docs: { description: "disallow unnecessary escape characters", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-useless-escape" }, diff --git a/lib/rules/no-useless-rename.js b/lib/rules/no-useless-rename.js index a7cec025da7..c0d27e600a8 100644 --- a/lib/rules/no-useless-rename.js +++ b/lib/rules/no-useless-rename.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "disallow renaming import, export, and destructured assignments to the same name", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/no-useless-rename" }, diff --git a/lib/rules/no-useless-return.js b/lib/rules/no-useless-return.js index 111cb21015f..87f05892a94 100644 --- a/lib/rules/no-useless-return.js +++ b/lib/rules/no-useless-return.js @@ -67,7 +67,6 @@ module.exports = { docs: { description: "disallow redundant return statements", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-useless-return" }, diff --git a/lib/rules/no-var.js b/lib/rules/no-var.js index f2cb96b1f70..a821c38a36e 100644 --- a/lib/rules/no-var.js +++ b/lib/rules/no-var.js @@ -185,7 +185,6 @@ module.exports = { docs: { description: "require `let` or `const` instead of `var`", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/no-var" }, diff --git a/lib/rules/no-void.js b/lib/rules/no-void.js index 99c83785552..dba4932385d 100644 --- a/lib/rules/no-void.js +++ b/lib/rules/no-void.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "disallow `void` operators", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-void" }, diff --git a/lib/rules/no-warning-comments.js b/lib/rules/no-warning-comments.js index e5f702bc7d7..23e3da35e14 100644 --- a/lib/rules/no-warning-comments.js +++ b/lib/rules/no-warning-comments.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "disallow specified warning terms in comments", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/no-warning-comments" }, diff --git a/lib/rules/no-whitespace-before-property.js b/lib/rules/no-whitespace-before-property.js index 226f873c5f6..9a492997d76 100644 --- a/lib/rules/no-whitespace-before-property.js +++ b/lib/rules/no-whitespace-before-property.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "disallow whitespace before properties", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/no-whitespace-before-property" }, diff --git a/lib/rules/no-with.js b/lib/rules/no-with.js index d3e52e02f3d..219a68094cb 100644 --- a/lib/rules/no-with.js +++ b/lib/rules/no-with.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "disallow `with` statements", - category: "Best Practices", recommended: true, url: "https://eslint.org/docs/rules/no-with" }, diff --git a/lib/rules/nonblock-statement-body-position.js b/lib/rules/nonblock-statement-body-position.js index 34e6eeac39d..7ed541b3802 100644 --- a/lib/rules/nonblock-statement-body-position.js +++ b/lib/rules/nonblock-statement-body-position.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce the location of single-line statements", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/nonblock-statement-body-position" }, diff --git a/lib/rules/object-curly-newline.js b/lib/rules/object-curly-newline.js index 1fbea00c5d7..e1609913582 100644 --- a/lib/rules/object-curly-newline.js +++ b/lib/rules/object-curly-newline.js @@ -150,7 +150,6 @@ module.exports = { docs: { description: "enforce consistent line breaks after opening and before closing braces", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/object-curly-newline" }, diff --git a/lib/rules/object-curly-spacing.js b/lib/rules/object-curly-spacing.js index c0044f5033c..b18ef574045 100644 --- a/lib/rules/object-curly-spacing.js +++ b/lib/rules/object-curly-spacing.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce consistent spacing inside braces", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/object-curly-spacing" }, diff --git a/lib/rules/object-property-newline.js b/lib/rules/object-property-newline.js index 0c7f800282e..7cca23ff31f 100644 --- a/lib/rules/object-property-newline.js +++ b/lib/rules/object-property-newline.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce placing object properties on separate lines", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/object-property-newline" }, diff --git a/lib/rules/object-shorthand.js b/lib/rules/object-shorthand.js index d78b1a71dfb..10bb07bbb0c 100644 --- a/lib/rules/object-shorthand.js +++ b/lib/rules/object-shorthand.js @@ -28,7 +28,6 @@ module.exports = { docs: { description: "require or disallow method and property shorthand syntax for object literals", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/object-shorthand" }, diff --git a/lib/rules/one-var-declaration-per-line.js b/lib/rules/one-var-declaration-per-line.js index db4674760a0..c0ad70004e6 100644 --- a/lib/rules/one-var-declaration-per-line.js +++ b/lib/rules/one-var-declaration-per-line.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "require or disallow newlines around variable declarations", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/one-var-declaration-per-line" }, diff --git a/lib/rules/one-var.js b/lib/rules/one-var.js index 3f338a48aaa..9c78ef87da8 100644 --- a/lib/rules/one-var.js +++ b/lib/rules/one-var.js @@ -34,7 +34,6 @@ module.exports = { docs: { description: "enforce variables to be declared either together or separately in functions", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/one-var" }, diff --git a/lib/rules/operator-assignment.js b/lib/rules/operator-assignment.js index 17cb4debfd8..34bdb861f46 100644 --- a/lib/rules/operator-assignment.js +++ b/lib/rules/operator-assignment.js @@ -63,7 +63,6 @@ module.exports = { docs: { description: "require or disallow assignment operator shorthand where possible", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/operator-assignment" }, diff --git a/lib/rules/operator-linebreak.js b/lib/rules/operator-linebreak.js index 2a0bfbddf17..6eab0cccbb4 100644 --- a/lib/rules/operator-linebreak.js +++ b/lib/rules/operator-linebreak.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "enforce consistent linebreak style for operators", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/operator-linebreak" }, diff --git a/lib/rules/padded-blocks.js b/lib/rules/padded-blocks.js index d999704dec4..ca83f11fc50 100644 --- a/lib/rules/padded-blocks.js +++ b/lib/rules/padded-blocks.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "require or disallow padding within blocks", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/padded-blocks" }, diff --git a/lib/rules/padding-line-between-statements.js b/lib/rules/padding-line-between-statements.js index c97b9956b71..073940a40eb 100644 --- a/lib/rules/padding-line-between-statements.js +++ b/lib/rules/padding-line-between-statements.js @@ -431,7 +431,6 @@ module.exports = { docs: { description: "require or disallow padding lines between statements", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/padding-line-between-statements" }, diff --git a/lib/rules/prefer-arrow-callback.js b/lib/rules/prefer-arrow-callback.js index 599a3266e2d..518bf4b2da3 100644 --- a/lib/rules/prefer-arrow-callback.js +++ b/lib/rules/prefer-arrow-callback.js @@ -151,7 +151,6 @@ module.exports = { docs: { description: "require using arrow functions for callbacks", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/prefer-arrow-callback" }, diff --git a/lib/rules/prefer-const.js b/lib/rules/prefer-const.js index f6e79e71c3e..b44bd7592b5 100644 --- a/lib/rules/prefer-const.js +++ b/lib/rules/prefer-const.js @@ -332,7 +332,6 @@ module.exports = { docs: { description: "require `const` declarations for variables that are never reassigned after declared", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/prefer-const" }, diff --git a/lib/rules/prefer-destructuring.js b/lib/rules/prefer-destructuring.js index 8cde3a47d5d..46986d237df 100644 --- a/lib/rules/prefer-destructuring.js +++ b/lib/rules/prefer-destructuring.js @@ -26,7 +26,6 @@ module.exports = { docs: { description: "require destructuring from arrays and/or objects", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/prefer-destructuring" }, diff --git a/lib/rules/prefer-exponentiation-operator.js b/lib/rules/prefer-exponentiation-operator.js index 6121af88971..de802ce1e11 100644 --- a/lib/rules/prefer-exponentiation-operator.js +++ b/lib/rules/prefer-exponentiation-operator.js @@ -90,7 +90,6 @@ module.exports = { docs: { description: "disallow the use of `Math.pow` in favor of the `**` operator", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/prefer-exponentiation-operator" }, diff --git a/lib/rules/prefer-named-capture-group.js b/lib/rules/prefer-named-capture-group.js index 7d0aa3f9dc0..41aa549327d 100644 --- a/lib/rules/prefer-named-capture-group.js +++ b/lib/rules/prefer-named-capture-group.js @@ -33,7 +33,6 @@ module.exports = { docs: { description: "enforce using named capture group in regular expression", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/prefer-named-capture-group" }, diff --git a/lib/rules/prefer-numeric-literals.js b/lib/rules/prefer-numeric-literals.js index cc82e6653c0..91bb26724eb 100644 --- a/lib/rules/prefer-numeric-literals.js +++ b/lib/rules/prefer-numeric-literals.js @@ -45,7 +45,6 @@ module.exports = { docs: { description: "disallow `parseInt()` and `Number.parseInt()` in favor of binary, octal, and hexadecimal literals", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/prefer-numeric-literals" }, diff --git a/lib/rules/prefer-object-spread.js b/lib/rules/prefer-object-spread.js index ab252c73ae3..3958a51b30e 100644 --- a/lib/rules/prefer-object-spread.js +++ b/lib/rules/prefer-object-spread.js @@ -247,7 +247,6 @@ module.exports = { docs: { description: "disallow using Object.assign with an object literal as the first argument and prefer the use of object spread instead.", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/prefer-object-spread" }, diff --git a/lib/rules/prefer-promise-reject-errors.js b/lib/rules/prefer-promise-reject-errors.js index ec16e445555..bdc1fef4d7c 100644 --- a/lib/rules/prefer-promise-reject-errors.js +++ b/lib/rules/prefer-promise-reject-errors.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "require using Error objects as Promise rejection reasons", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/prefer-promise-reject-errors" }, diff --git a/lib/rules/prefer-reflect.js b/lib/rules/prefer-reflect.js index 156d61251c4..fea88c67827 100644 --- a/lib/rules/prefer-reflect.js +++ b/lib/rules/prefer-reflect.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require `Reflect` methods where applicable", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/prefer-reflect" }, diff --git a/lib/rules/prefer-regex-literals.js b/lib/rules/prefer-regex-literals.js index 9e8ce023547..fbfeb5636d7 100644 --- a/lib/rules/prefer-regex-literals.js +++ b/lib/rules/prefer-regex-literals.js @@ -54,7 +54,6 @@ module.exports = { docs: { description: "disallow use of the `RegExp` constructor in favor of regular expression literals", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/prefer-regex-literals" }, diff --git a/lib/rules/prefer-rest-params.js b/lib/rules/prefer-rest-params.js index 3ecea732af8..157f0bbd61a 100644 --- a/lib/rules/prefer-rest-params.js +++ b/lib/rules/prefer-rest-params.js @@ -65,7 +65,6 @@ module.exports = { docs: { description: "require rest parameters instead of `arguments`", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/prefer-rest-params" }, diff --git a/lib/rules/prefer-spread.js b/lib/rules/prefer-spread.js index d3c3c4d2297..3944fedb126 100644 --- a/lib/rules/prefer-spread.js +++ b/lib/rules/prefer-spread.js @@ -49,7 +49,6 @@ module.exports = { docs: { description: "require spread operators instead of `.apply()`", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/prefer-spread" }, diff --git a/lib/rules/prefer-template.js b/lib/rules/prefer-template.js index cb967660a62..564dd555f8e 100644 --- a/lib/rules/prefer-template.js +++ b/lib/rules/prefer-template.js @@ -128,7 +128,6 @@ module.exports = { docs: { description: "require template literals instead of string concatenation", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/prefer-template" }, diff --git a/lib/rules/quote-props.js b/lib/rules/quote-props.js index 741acf8db9f..ce277cdcc9d 100644 --- a/lib/rules/quote-props.js +++ b/lib/rules/quote-props.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "require quotes around object literal property names", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/quote-props" }, diff --git a/lib/rules/quotes.js b/lib/rules/quotes.js index 54946ff7814..d7959c0d1f0 100644 --- a/lib/rules/quotes.js +++ b/lib/rules/quotes.js @@ -80,7 +80,6 @@ module.exports = { docs: { description: "enforce the consistent use of either backticks, double, or single quotes", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/quotes" }, diff --git a/lib/rules/radix.js b/lib/rules/radix.js index f1a7c1af265..0c6c6ffb1a6 100644 --- a/lib/rules/radix.js +++ b/lib/rules/radix.js @@ -80,7 +80,6 @@ module.exports = { docs: { description: "enforce the consistent use of the radix argument when using `parseInt()`", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/radix" }, diff --git a/lib/rules/require-atomic-updates.js b/lib/rules/require-atomic-updates.js index 8a315a8793a..b25f150f6e0 100644 --- a/lib/rules/require-atomic-updates.js +++ b/lib/rules/require-atomic-updates.js @@ -171,7 +171,6 @@ module.exports = { docs: { description: "disallow assignments that can lead to race conditions due to usage of `await` or `yield`", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/require-atomic-updates" }, diff --git a/lib/rules/require-await.js b/lib/rules/require-await.js index 9b5acc78c81..8ec6f547ae9 100644 --- a/lib/rules/require-await.js +++ b/lib/rules/require-await.js @@ -34,7 +34,6 @@ module.exports = { docs: { description: "disallow async functions which have no `await` expression", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/require-await" }, diff --git a/lib/rules/require-jsdoc.js b/lib/rules/require-jsdoc.js index e581b2bee46..f1c21688d48 100644 --- a/lib/rules/require-jsdoc.js +++ b/lib/rules/require-jsdoc.js @@ -10,7 +10,6 @@ module.exports = { docs: { description: "require JSDoc comments", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/require-jsdoc" }, diff --git a/lib/rules/require-unicode-regexp.js b/lib/rules/require-unicode-regexp.js index 880405e9a25..a332b48da66 100644 --- a/lib/rules/require-unicode-regexp.js +++ b/lib/rules/require-unicode-regexp.js @@ -26,7 +26,6 @@ module.exports = { docs: { description: "enforce the use of `u` flag on RegExp", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/require-unicode-regexp" }, diff --git a/lib/rules/require-yield.js b/lib/rules/require-yield.js index af2344dfa6b..f5b5d530396 100644 --- a/lib/rules/require-yield.js +++ b/lib/rules/require-yield.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require generator functions to contain `yield`", - category: "ECMAScript 6", recommended: true, url: "https://eslint.org/docs/rules/require-yield" }, diff --git a/lib/rules/rest-spread-spacing.js b/lib/rules/rest-spread-spacing.js index 8cb9814f0c9..a636defdcd9 100644 --- a/lib/rules/rest-spread-spacing.js +++ b/lib/rules/rest-spread-spacing.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce spacing between rest and spread operators and their expressions", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/rest-spread-spacing" }, diff --git a/lib/rules/semi-spacing.js b/lib/rules/semi-spacing.js index e9f2ceb5c8c..e5e2ae25cc7 100644 --- a/lib/rules/semi-spacing.js +++ b/lib/rules/semi-spacing.js @@ -17,7 +17,6 @@ module.exports = { docs: { description: "enforce consistent spacing before and after semicolons", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/semi-spacing" }, diff --git a/lib/rules/semi-style.js b/lib/rules/semi-style.js index 186f5c0d6ec..43d8d51f969 100644 --- a/lib/rules/semi-style.js +++ b/lib/rules/semi-style.js @@ -67,7 +67,6 @@ module.exports = { docs: { description: "enforce location of semicolons", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/semi-style" }, diff --git a/lib/rules/semi.js b/lib/rules/semi.js index 87086e981b0..d79b22bc876 100644 --- a/lib/rules/semi.js +++ b/lib/rules/semi.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "require or disallow semicolons instead of ASI", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/semi" }, diff --git a/lib/rules/sort-imports.js b/lib/rules/sort-imports.js index 4c3ddec7669..dd43daddf05 100644 --- a/lib/rules/sort-imports.js +++ b/lib/rules/sort-imports.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "enforce sorted import declarations within modules", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/sort-imports" }, diff --git a/lib/rules/sort-keys.js b/lib/rules/sort-keys.js index 8a95ee25d61..65a99142279 100644 --- a/lib/rules/sort-keys.js +++ b/lib/rules/sort-keys.js @@ -81,7 +81,6 @@ module.exports = { docs: { description: "require object keys to be sorted", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/sort-keys" }, diff --git a/lib/rules/sort-vars.js b/lib/rules/sort-vars.js index 7add2cf74b2..0616c44ac62 100644 --- a/lib/rules/sort-vars.js +++ b/lib/rules/sort-vars.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require variables within the same declaration block to be sorted", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/sort-vars" }, diff --git a/lib/rules/space-before-blocks.js b/lib/rules/space-before-blocks.js index 87ef9bfd880..45af9a56cc1 100644 --- a/lib/rules/space-before-blocks.js +++ b/lib/rules/space-before-blocks.js @@ -40,7 +40,6 @@ module.exports = { docs: { description: "enforce consistent spacing before blocks", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/space-before-blocks" }, diff --git a/lib/rules/space-before-function-paren.js b/lib/rules/space-before-function-paren.js index 1021a110cfd..b60bee04097 100644 --- a/lib/rules/space-before-function-paren.js +++ b/lib/rules/space-before-function-paren.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "enforce consistent spacing before `function` definition opening parenthesis", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/space-before-function-paren" }, diff --git a/lib/rules/space-in-parens.js b/lib/rules/space-in-parens.js index b0a604d955d..24378b89f04 100644 --- a/lib/rules/space-in-parens.js +++ b/lib/rules/space-in-parens.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "enforce consistent spacing inside parentheses", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/space-in-parens" }, diff --git a/lib/rules/space-infix-ops.js b/lib/rules/space-infix-ops.js index b0511e13cba..8065b5240b9 100644 --- a/lib/rules/space-infix-ops.js +++ b/lib/rules/space-infix-ops.js @@ -16,7 +16,6 @@ module.exports = { docs: { description: "require spacing around infix operators", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/space-infix-ops" }, diff --git a/lib/rules/space-unary-ops.js b/lib/rules/space-unary-ops.js index 57f6e784501..de9018f9b69 100644 --- a/lib/rules/space-unary-ops.js +++ b/lib/rules/space-unary-ops.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "enforce consistent spacing before or after unary operators", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/space-unary-ops" }, diff --git a/lib/rules/spaced-comment.js b/lib/rules/spaced-comment.js index 226a2d44798..6f0b432b765 100644 --- a/lib/rules/spaced-comment.js +++ b/lib/rules/spaced-comment.js @@ -152,7 +152,6 @@ module.exports = { docs: { description: "enforce consistent spacing after the `//` or `/*` in a comment", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/spaced-comment" }, diff --git a/lib/rules/strict.js b/lib/rules/strict.js index b0d6cf9172a..24af39dabee 100644 --- a/lib/rules/strict.js +++ b/lib/rules/strict.js @@ -69,7 +69,6 @@ module.exports = { docs: { description: "require or disallow strict mode directives", - category: "Strict Mode", recommended: false, url: "https://eslint.org/docs/rules/strict" }, diff --git a/lib/rules/switch-colon-spacing.js b/lib/rules/switch-colon-spacing.js index c90641573c6..a218a34f4ac 100644 --- a/lib/rules/switch-colon-spacing.js +++ b/lib/rules/switch-colon-spacing.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "enforce spacing around colons of switch statements", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/switch-colon-spacing" }, diff --git a/lib/rules/symbol-description.js b/lib/rules/symbol-description.js index 155cea4dc0b..9f5d9358f00 100644 --- a/lib/rules/symbol-description.js +++ b/lib/rules/symbol-description.js @@ -22,7 +22,6 @@ module.exports = { docs: { description: "require symbol descriptions", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/symbol-description" }, diff --git a/lib/rules/template-curly-spacing.js b/lib/rules/template-curly-spacing.js index 26043bc9122..5133a54539c 100644 --- a/lib/rules/template-curly-spacing.js +++ b/lib/rules/template-curly-spacing.js @@ -21,7 +21,6 @@ module.exports = { docs: { description: "require or disallow spacing around embedded expressions of template strings", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/template-curly-spacing" }, diff --git a/lib/rules/template-tag-spacing.js b/lib/rules/template-tag-spacing.js index 16f586255af..45b66068a83 100644 --- a/lib/rules/template-tag-spacing.js +++ b/lib/rules/template-tag-spacing.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require or disallow spacing between template tags and their literals", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/template-tag-spacing" }, diff --git a/lib/rules/unicode-bom.js b/lib/rules/unicode-bom.js index 39642f85193..e80497d19ca 100644 --- a/lib/rules/unicode-bom.js +++ b/lib/rules/unicode-bom.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "require or disallow Unicode byte order mark (BOM)", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/unicode-bom" }, diff --git a/lib/rules/use-isnan.js b/lib/rules/use-isnan.js index ef95b21314a..cd8331f932e 100644 --- a/lib/rules/use-isnan.js +++ b/lib/rules/use-isnan.js @@ -37,7 +37,6 @@ module.exports = { docs: { description: "require calls to `isNaN()` when checking for `NaN`", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/use-isnan" }, diff --git a/lib/rules/valid-jsdoc.js b/lib/rules/valid-jsdoc.js index 9ec6938e35a..02abd7065d9 100644 --- a/lib/rules/valid-jsdoc.js +++ b/lib/rules/valid-jsdoc.js @@ -20,7 +20,6 @@ module.exports = { docs: { description: "enforce valid JSDoc comments", - category: "Possible Errors", recommended: false, url: "https://eslint.org/docs/rules/valid-jsdoc" }, diff --git a/lib/rules/valid-typeof.js b/lib/rules/valid-typeof.js index a0f20f74d0e..33b64f56cc3 100644 --- a/lib/rules/valid-typeof.js +++ b/lib/rules/valid-typeof.js @@ -14,7 +14,6 @@ module.exports = { docs: { description: "enforce comparing `typeof` expressions against valid strings", - category: "Possible Errors", recommended: true, url: "https://eslint.org/docs/rules/valid-typeof" }, diff --git a/lib/rules/vars-on-top.js b/lib/rules/vars-on-top.js index 592e9be4aa1..6f913dcad9d 100644 --- a/lib/rules/vars-on-top.js +++ b/lib/rules/vars-on-top.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require `var` declarations be placed at the top of their containing scope", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/vars-on-top" }, diff --git a/lib/rules/wrap-iife.js b/lib/rules/wrap-iife.js index 07e5b84a4a5..498d7bd2842 100644 --- a/lib/rules/wrap-iife.js +++ b/lib/rules/wrap-iife.js @@ -43,7 +43,6 @@ module.exports = { docs: { description: "require parentheses around immediate `function` invocations", - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/wrap-iife" }, diff --git a/lib/rules/wrap-regex.js b/lib/rules/wrap-regex.js index 4ecbcecbbeb..945eb5ecea1 100644 --- a/lib/rules/wrap-regex.js +++ b/lib/rules/wrap-regex.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require parenthesis around regex literals", - category: "Stylistic Issues", recommended: false, url: "https://eslint.org/docs/rules/wrap-regex" }, diff --git a/lib/rules/yield-star-spacing.js b/lib/rules/yield-star-spacing.js index 20b8e9ea91e..8c3eefa4acd 100644 --- a/lib/rules/yield-star-spacing.js +++ b/lib/rules/yield-star-spacing.js @@ -15,7 +15,6 @@ module.exports = { docs: { description: "require or disallow spacing around the `*` in `yield*` expressions", - category: "ECMAScript 6", recommended: false, url: "https://eslint.org/docs/rules/yield-star-spacing" }, diff --git a/lib/rules/yoda.js b/lib/rules/yoda.js index 2fca7571113..2d018dc3df1 100644 --- a/lib/rules/yoda.js +++ b/lib/rules/yoda.js @@ -121,7 +121,6 @@ module.exports = { docs: { description: 'require or disallow "Yoda" conditions', - category: "Best Practices", recommended: false, url: "https://eslint.org/docs/rules/yoda" }, diff --git a/tests/tools/internal-rules/no-invalid-meta.js b/tests/tools/internal-rules/no-invalid-meta.js index 6db173a15a1..5013e4bdbe3 100644 --- a/tests/tools/internal-rules/no-invalid-meta.js +++ b/tests/tools/internal-rules/no-invalid-meta.js @@ -27,7 +27,6 @@ ruleTester.run("no-invalid-meta", rule, { " meta: {", " docs: {", " description: 'some rule',", - " category: 'Internal',", " recommended: false", " },", " schema: []", @@ -51,7 +50,6 @@ ruleTester.run("no-invalid-meta", rule, { " meta: {", " docs: {", " description: 'some rule',", - " category: 'Internal',", " recommended: false", " },", " schema: []", @@ -73,7 +71,6 @@ ruleTester.run("no-invalid-meta", rule, { " meta: {", " docs: {", " description: 'some rule',", - " category: 'Internal',", " recommended: false", " },", " schema: [],", @@ -156,31 +153,6 @@ ruleTester.run("no-invalid-meta", rule, { " meta: {", " docs: {", " description: 'some rule',", - " recommended: false", - " },", - " schema: []", - " },", - - " create: function(context) {", - " return {", - " Program: function(node) {}", - " };", - " }", - "};" - ].join("\n"), - errors: [{ - messageId: "missingMetaDocsCategory", - line: 2, - column: 5 - }] - }, - { - code: [ - "module.exports = {", - " meta: {", - " docs: {", - " description: 'some rule',", - " category: 'Internal'", " },", " schema: []", " },", diff --git a/tools/internal-rules/no-invalid-meta.js b/tools/internal-rules/no-invalid-meta.js index 69b43d4c883..2bfa2e21a4a 100644 --- a/tools/internal-rules/no-invalid-meta.js +++ b/tools/internal-rules/no-invalid-meta.js @@ -50,17 +50,6 @@ function hasMetaDocs(metaPropertyNode) { return Boolean(getPropertyFromObject("docs", metaPropertyNode.value)); } -/** - * Whether this `meta` ObjectExpression has a `docs.category` property defined or not. - * @param {ASTNode} metaPropertyNode The `meta` ObjectExpression for this rule. - * @returns {boolean} `true` if a `docs.category` property exists. - */ -function hasMetaDocsCategory(metaPropertyNode) { - const metaDocs = getPropertyFromObject("docs", metaPropertyNode.value); - - return metaDocs && getPropertyFromObject("category", metaDocs.value); -} - /** * Whether this `meta` ObjectExpression has a `docs.recommended` property defined or not. * @param {ASTNode} metaPropertyNode The `meta` ObjectExpression for this rule. @@ -91,11 +80,6 @@ function checkMetaValidity(context, exportsNode) { return; } - if (!hasMetaDocsCategory(metaProperty)) { - context.report({ node: metaProperty, messageId: "missingMetaDocsCategory" }); - return; - } - if (!hasMetaDocsRecommended(metaProperty)) { context.report({ node: metaProperty, messageId: "missingMetaDocsRecommended" }); } @@ -109,7 +93,6 @@ module.exports = { meta: { docs: { description: "enforce correct use of `meta` property in core rules", - category: "Internal", recommended: false }, type: "problem", @@ -117,7 +100,6 @@ module.exports = { messages: { missingMeta: "Rule is missing a meta property.", missingMetaDocs: "Rule is missing a meta.docs property.", - missingMetaDocsCategory: "Rule is missing a meta.docs.category property.", missingMetaDocsRecommended: "Rule is missing a meta.docs.recommended property.", noExport: "Rule does not export anything. Make sure rule exports an object according to new rule format." } diff --git a/tools/update-rule-types.js b/tools/update-rule-types.js index 0cb40a7b176..6a68586d6de 100644 --- a/tools/update-rule-types.js +++ b/tools/update-rule-types.js @@ -30,29 +30,11 @@ module.exports = (fileInfo, api) => { } const typeNode = metaNode.value.properties.find(node => node.key.name === "type"); - const docsNode = metaNode.value.properties.find(node => node.key.name === "docs"); - const categoryNode = docsNode.value.properties.find(node => node.key.name === "category").value; let ruleType; - // the rule-types.json file takes highest priority if (ruleName in ruleTypes) { ruleType = ruleTypes[ruleName]; - } else { - - // otherwise fallback to category - switch (categoryNode.value) { - case "Stylistic Issues": - ruleType = "style"; - break; - - case "Possible Errors": - ruleType = "problem"; - break; - - default: - ruleType = "suggestion"; - } } if (typeNode) {