eslint 5.8.0 → 5.9.0

package/lib/rules/no-useless-rename.js

@@ -11,13 +11,17 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         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"
         },
+
         fixable: "code",
+
         schema: [
             {
                 type: "object",

package/lib/rules/no-useless-return.js

@@ -66,12 +66,15 @@ function isInFinally(node) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "disallow redundant return statements",
             category: "Best Practices",
             recommended: false,
             url: "https://eslint.org/docs/rules/no-useless-return"
         },
+
         fixable: "code",
         schema: []
     },

package/lib/rules/no-var.js

@@ -180,6 +180,8 @@ function hasReferenceInTDZ(node) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require `let` or `const` instead of `var`",
             category: "ECMAScript 6",

package/lib/rules/no-void.js

@@ -10,6 +10,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "disallow `void` operators",
             category: "Best Practices",

package/lib/rules/no-warning-comments.js

@@ -13,6 +13,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "disallow specified warning terms in comments",
             category: "Best Practices",

package/lib/rules/no-whitespace-before-property.js

@@ -16,6 +16,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "disallow whitespace before properties",
             category: "Stylistic Issues",

package/lib/rules/no-with.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "disallow `with` statements",
             category: "Best Practices",

package/lib/rules/nonblock-statement-body-position.js

@@ -12,13 +12,17 @@ const POSITION_SCHEMA = { enum: ["beside", "below", "any"] };
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce the location of single-line statements",
             category: "Stylistic Issues",
             recommended: false,
             url: "https://eslint.org/docs/rules/nonblock-statement-body-position"
         },
+
         fixable: "whitespace",
+
         schema: [
             POSITION_SCHEMA,
             {

package/lib/rules/object-curly-newline.js

@@ -134,13 +134,17 @@ function areLineBreaksRequired(node, options, first, last) {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent line breaks inside braces",
             category: "Stylistic Issues",
             recommended: false,
             url: "https://eslint.org/docs/rules/object-curly-newline"
         },
+
         fixable: "whitespace",
+
         schema: [
             {
                 oneOf: [

package/lib/rules/object-curly-spacing.js

@@ -12,6 +12,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent spacing inside braces",
             category: "Stylistic Issues",

package/lib/rules/object-property-newline.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce placing object properties on separate lines",
             category: "Stylistic Issues",
@@ -38,8 +40,7 @@ module.exports = {
 
     create(context) {
         const allowSameLine = context.options[0] && (
-            Boolean(context.options[0].allowAllPropertiesOnSameLine) ||
-            Boolean(context.options[0].allowMultiplePropertiesPerLine) // Deprecated
+            (Boolean(context.options[0].allowAllPropertiesOnSameLine) || Boolean(context.options[0].allowMultiplePropertiesPerLine)) // Deprecated
         );
         const errorMessage = allowSameLine
             ? "Object properties must go on a new line if they aren't all on the same line."

package/lib/rules/object-shorthand.js

@@ -24,6 +24,8 @@ const astUtils = require("../util/ast-utils");
 //------------------------------------------------------------------------------
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require or disallow method and property shorthand syntax for object literals",
             category: "ECMAScript 6",

package/lib/rules/one-var-declaration-per-line.js

@@ -10,6 +10,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require or disallow newlines around variable declarations",
             category: "Stylistic Issues",

package/lib/rules/one-var.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce variables to be declared either together or separately in functions",
             category: "Stylistic Issues",

package/lib/rules/operator-assignment.js

@@ -89,6 +89,8 @@ function canBeFixed(node) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require or disallow assignment operator shorthand where possible",
             category: "Stylistic Issues",

package/lib/rules/operator-linebreak.js

@@ -17,6 +17,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent linebreak style for operators",
             category: "Stylistic Issues",

package/lib/rules/padded-blocks.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require or disallow padding within blocks",
             category: "Stylistic Issues",

package/lib/rules/padding-line-between-statements.js

@@ -400,13 +400,17 @@ const StatementTypes = {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require or disallow padding lines between statements",
             category: "Stylistic Issues",
             recommended: false,
             url: "https://eslint.org/docs/rules/padding-line-between-statements"
         },
+
         fixable: "whitespace",
+
         schema: {
             definitions: {
                 paddingType: {

package/lib/rules/prefer-arrow-callback.js

@@ -132,6 +132,8 @@ function hasDuplicateParams(paramsList) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require using arrow functions for callbacks",
             category: "ECMAScript 6",

package/lib/rules/prefer-const.js

@@ -330,6 +330,8 @@ function findUp(node, type, shouldStop) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require `const` declarations for variables that are never reassigned after declared",
             category: "ECMAScript 6",
@@ -357,6 +359,8 @@ module.exports = {
         const shouldMatchAnyDestructuredVariable = options.destructuring !== "all";
         const ignoreReadBeforeAssign = options.ignoreReadBeforeAssign === true;
         const variables = [];
+        let reportCount = 0;
+        let name = "";
 
         /**
          * Reports given identifier nodes if all of the nodes should be declared
@@ -377,14 +381,41 @@ module.exports = {
 
             if (nodes.length && (shouldMatchAnyDestructuredVariable || nodesToReport.length === nodes.length)) {
                 const varDeclParent = findUp(nodes[0], "VariableDeclaration", parentNode => parentNode.type.endsWith("Statement"));
-                const shouldFix = varDeclParent &&
+                const isVarDecParentNull = varDeclParent === null;
+
+                if (!isVarDecParentNull && varDeclParent.declarations.length > 0) {
+                    const firstDeclaration = varDeclParent.declarations[0];
+
+                    if (firstDeclaration.init) {
+                        const firstDecParent = firstDeclaration.init.parent;
+
+                        /*
+                         * First we check the declaration type and then depending on
+                         * if the type is a "VariableDeclarator" or its an "ObjectPattern"
+                         * we compare the name from the first identifier, if the names are different
+                         * we assign the new name and reset the count of reportCount and nodeCount in
+                         * order to check each block for the number of reported errors and base our fix
+                         * based on comparing nodes.length and nodesToReport.length.
+                         */
+
+                        if (firstDecParent.type === "VariableDeclarator") {
+
+                            if (firstDecParent.id.name !== name) {
+                                name = firstDecParent.id.name;
+                                reportCount = 0;
+                            }
+
+                            if (firstDecParent.id.type === "ObjectPattern") {
+                                if (firstDecParent.init.name !== name) {
+                                    name = firstDecParent.init.name;
+                                    reportCount = 0;
+                                }
+                            }
+                        }
+                    }
+                }
 
-                    /*
-                     * If there are multiple variable declarations, like {let a = 1, b = 2}, then
-                     * do not attempt to fix if one of the declarations should be `const`. It's
-                     * too hard to know how the developer would want to automatically resolve the issue.
-                     */
-                    varDeclParent.declarations.length === 1 &&
+                let shouldFix = varDeclParent &&
 
                     // Don't do a fix unless the variable is initialized (or it's in a for-in or for-of loop)
                     (varDeclParent.parent.type === "ForInStatement" || varDeclParent.parent.type === "ForOfStatement" || varDeclParent.declarations[0].init) &&
@@ -396,6 +427,21 @@ module.exports = {
                      */
                     nodesToReport.length === nodes.length;
 
+                if (!isVarDecParentNull && varDeclParent.declarations && varDeclParent.declarations.length !== 1) {
+
+                    if (varDeclParent && varDeclParent.declarations && varDeclParent.declarations.length >= 1) {
+
+                        /*
+                         * Add nodesToReport.length to a count, then comparing the count to the length
+                         * of the declarations in the current block.
+                         */
+
+                        reportCount += nodesToReport.length;
+
+                        shouldFix = shouldFix && (reportCount === varDeclParent.declarations.length);
+                    }
+                }
+
                 nodesToReport.forEach(node => {
                     context.report({
                         node,

package/lib/rules/prefer-destructuring.js

@@ -10,12 +10,15 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require destructuring from arrays and/or objects",
             category: "ECMAScript 6",
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-destructuring"
         },
+
         schema: [
             {
 

package/lib/rules/prefer-numeric-literals.js

@@ -38,6 +38,8 @@ function isParseInt(calleeNode) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "disallow `parseInt()` and `Number.parseInt()` in favor of binary, octal, and hexadecimal literals",
             category: "ECMAScript 6",
@@ -46,7 +48,6 @@ module.exports = {
         },
 
         schema: [],
-
         fixable: "code"
     },
 

package/lib/rules/prefer-object-spread.js

@@ -212,6 +212,8 @@ function defineFixer(node, sourceCode) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description:
                 "disallow using Object.assign with an object literal as the first argument and prefer the use of object spread instead.",
@@ -219,8 +221,10 @@ module.exports = {
             recommended: false,
             url: "https://eslint.org/docs/rules/prefer-object-spread"
         },
+
         schema: [],
         fixable: "code",
+
         messages: {
             useSpreadMessage: "Use an object spread instead of `Object.assign` eg: `{ ...foo }`",
             useLiteralMessage: "Use an object literal instead of `Object.assign`. eg: `{ foo: bar }`"

package/lib/rules/prefer-promise-reject-errors.js

@@ -12,13 +12,17 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         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"
         },
+
         fixable: null,
+
         schema: [
             {
                 type: "object",

package/lib/rules/prefer-reflect.js

@@ -11,16 +11,19 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require `Reflect` methods where applicable",
             category: "ECMAScript 6",
             recommended: false,
-            replacedBy: [],
             url: "https://eslint.org/docs/rules/prefer-reflect"
         },
 
         deprecated: true,
 
+        replacedBy: [],
+
         schema: [
             {
                 type: "object",

package/lib/rules/prefer-rest-params.js

@@ -62,6 +62,8 @@ function isNotNormalMemberAccess(reference) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require rest parameters instead of `arguments`",
             category: "ECMAScript 6",

package/lib/rules/prefer-spread.js

@@ -49,6 +49,8 @@ function isValidThisArg(expectedThis, thisArg, context) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require spread operators instead of `.apply()`",
             category: "ECMAScript 6",
@@ -57,7 +59,6 @@ module.exports = {
         },
 
         schema: [],
-
         fixable: "code"
     },
 

package/lib/rules/prefer-template.js

@@ -141,6 +141,8 @@ function endsWithTemplateCurly(node) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require template literals instead of string concatenation",
             category: "ECMAScript 6",
@@ -149,7 +151,6 @@ module.exports = {
         },
 
         schema: [],
-
         fixable: "code"
     },
 

package/lib/rules/quote-props.js

@@ -17,6 +17,8 @@ const espree = require("espree"),
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require quotes around object literal property names",
             category: "Stylistic Issues",

package/lib/rules/quotes.js

@@ -76,6 +76,8 @@ const AVOID_ESCAPE = "avoid-escape";
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce the consistent use of either backticks, double, or single quotes",
             category: "Stylistic Issues",

package/lib/rules/radix.js

@@ -78,6 +78,8 @@ function isDefaultRadix(radix) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce the consistent use of the radix argument when using `parseInt()`",
             category: "Best Practices",

package/lib/rules/require-atomic-updates.js

@@ -12,14 +12,18 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "problem",
+
         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"
         },
+
         fixable: null,
         schema: [],
+
         messages: {
             nonAtomicUpdate: "Possible race condition: `{{value}}` might be reassigned based on an outdated value of `{{value}}`."
         }

package/lib/rules/require-await.js

@@ -31,12 +31,15 @@ function capitalizeFirstLetter(text) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "disallow async functions which have no `await` expression",
             category: "Best Practices",
             recommended: false,
             url: "https://eslint.org/docs/rules/require-await"
         },
+
         schema: []
     },
 

package/lib/rules/require-jsdoc.js

@@ -6,6 +6,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require JSDoc comments",
             category: "Stylistic Issues",

package/lib/rules/require-unicode-regexp.js

@@ -22,15 +22,19 @@ const {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce the use of `u` flag on RegExp",
             category: "Best Practices",
             recommended: false,
             url: "https://eslint.org/docs/rules/require-unicode-regexp"
         },
+
         messages: {
             requireUFlag: "Use the 'u' flag."
         },
+
         schema: []
     },
 

package/lib/rules/require-yield.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require generator functions to contain `yield`",
             category: "ECMAScript 6",

package/lib/rules/rest-spread-spacing.js

@@ -11,13 +11,17 @@
 
 module.exports = {
     meta: {
+        type: "layout",
+
         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"
         },
+
         fixable: "whitespace",
+
         schema: [
             {
                 enum: ["always", "never"]

package/lib/rules/semi-spacing.js

@@ -13,6 +13,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent spacing before and after semicolons",
             category: "Stylistic Issues",

package/lib/rules/semi-style.js

@@ -65,12 +65,15 @@ function isLastChild(node) {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce location of semicolons",
             category: "Stylistic Issues",
             recommended: false,
             url: "https://eslint.org/docs/rules/semi-style"
         },
+
         schema: [{ enum: ["last", "first"] }],
         fixable: "whitespace"
     },

package/lib/rules/semi.js

@@ -17,6 +17,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require or disallow semicolons instead of ASI",
             category: "Stylistic Issues",

package/lib/rules/sort-imports.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce sorted import declarations within modules",
             category: "ECMAScript 6",