Breaking: update eslint:recommended (fixes #5103)

conf/eslint.json

@@ -26,7 +26,7 @@
         "no-else-return": 0,
         "no-empty": 2,
         "no-empty-character-class": 2,
-        "no-empty-label": 2,
+        "no-empty-label": 0,
         "no-empty-pattern": 2,
         "no-eq-null": 0,
         "no-eval": 0,
@@ -85,7 +85,7 @@
         "no-restricted-syntax": 0,
         "no-return-assign": 0,
         "no-script-url": 0,
-        "no-self-assign": 0,
+        "no-self-assign": 2,
         "no-self-compare": 0,
         "no-sequences": 0,
         "no-shadow": 0,
@@ -107,7 +107,7 @@
         "no-unneeded-ternary": 0,
         "no-unreachable": 2,
         "no-unused-expressions": 0,
-        "no-unused-labels": 0,
+        "no-unused-labels": 2,
         "no-unused-vars": 2,
         "no-use-before-define": 0,
         "no-useless-call": 0,

docs/rules/README.md

@@ -57,7 +57,7 @@ These are rules designed to prevent you from making mistakes. They either prescr
 * [no-case-declarations](no-case-declarations.md) - disallow lexical declarations in case clauses (recommended)
 * [no-div-regex](no-div-regex.md) - disallow division operators explicitly at beginning of regular expression
 * [no-else-return](no-else-return.md) - disallow `else` after a `return` in an `if`
-* [no-empty-label](no-empty-label.md) - disallow use of labels for anything other than loops and switches (recommended)
+* [no-empty-label](no-empty-label.md) - disallow use of labels for anything other than loops and switches
 * [no-empty-pattern](no-empty-pattern.md) - disallow use of empty destructuring patterns (recommended)
 * [no-eq-null](no-eq-null.md) - disallow comparisons to null without a type-checking operator
 * [no-eval](no-eval.md) - disallow use of `eval()`
@@ -88,13 +88,13 @@ These are rules designed to prevent you from making mistakes. They either prescr
 * [no-redeclare](no-redeclare.md) - disallow declaring the same variable more than once (recommended)
 * [no-return-assign](no-return-assign.md) - disallow use of assignment in `return` statement
 * [no-script-url](no-script-url.md) - disallow use of `javascript:` urls.
-* [no-self-assign](no-self-assign.md) - disallow assignments where both sides are exactly the same
+* [no-self-assign](no-self-assign.md) - disallow assignments where both sides are exactly the same (recommended)
 * [no-self-compare](no-self-compare.md) - disallow comparisons where both sides are exactly the same
 * [no-sequences](no-sequences.md) - disallow use of the comma operator
 * [no-throw-literal](no-throw-literal.md) - restrict what can be thrown as an exception
 * [no-unmodified-loop-condition](no-unmodified-loop-condition.md) - disallow unmodified conditions of loops
 * [no-unused-expressions](no-unused-expressions.md) - disallow usage of expressions in statement position
-* [no-unused-labels](no-unused-labels.md) - disallow unused labels
+* [no-unused-labels](no-unused-labels.md) - disallow unused labels (recommended)
 * [no-useless-call](no-useless-call.md) - disallow unnecessary `.call()` and `.apply()`
 * [no-useless-concat](no-useless-concat.md) - disallow unnecessary concatenation of literals or template literals
 * [no-void](no-void.md) - disallow use of the `void` operator