tools: update ESLint to 6.6.0

Update ESLint to 6.6.0

PR-URL: https://github.com/nodejs/node/pull/30123
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

1 files changed, 22 insertions, 29 deletions

diff --git a/tools/node_modules/eslint/lib/rules/no-unused-vars.js b/tools/node_modules/eslint/lib/rules/no-unused-vars.js
index 8094de57c7..63e6e048ef 100644
--- a/tools/node_modules/eslint/lib/rules/no-unused-vars.js
+++ b/tools/node_modules/eslint/lib/rules/no-unused-vars.js
@@ -103,7 +103,7 @@ module.exports = {
         /**
          * Generate the warning message about the variable being
          * defined and unused, including the ignore pattern if configured.
-         * @param {Variable} unusedVar - eslint-scope variable object.
+         * @param {Variable} unusedVar eslint-scope variable object.
          * @returns {string} The warning message to be used with this unused variable.
          */
         function getDefinedMessage(unusedVar) {
@@ -146,7 +146,7 @@ module.exports = {
 
         /**
          * Determines if a given variable is being exported from a module.
-         * @param {Variable} variable - eslint-scope variable object.
+         * @param {Variable} variable eslint-scope variable object.
          * @returns {boolean} True if the variable is exported, false if not.
          * @private
          */
@@ -172,7 +172,7 @@ module.exports = {
 
         /**
          * Determines if a variable has a sibling rest property
-         * @param {Variable} variable - eslint-scope variable object.
+         * @param {Variable} variable eslint-scope variable object.
          * @returns {boolean} True if the variable is exported, false if not.
          * @private
          */
@@ -195,7 +195,7 @@ module.exports = {
 
         /**
          * Determines if a reference is a read operation.
-         * @param {Reference} ref - An eslint-scope Reference
+         * @param {Reference} ref An eslint-scope Reference
          * @returns {boolean} whether the given reference represents a read operation
          * @private
          */
@@ -205,8 +205,8 @@ module.exports = {
 
         /**
          * Determine if an identifier is referencing an enclosing function name.
-         * @param {Reference} ref - The reference to check.
-         * @param {ASTNode[]} nodes - The candidate function nodes.
+         * @param {Reference} ref The reference to check.
+         * @param {ASTNode[]} nodes The candidate function nodes.
          * @returns {boolean} True if it's a self-reference, false if not.
          * @private
          */
@@ -226,7 +226,7 @@ module.exports = {
 
         /**
          * Gets a list of function definitions for a specified variable.
-         * @param {Variable} variable - eslint-scope variable object.
+         * @param {Variable} variable eslint-scope variable object.
          * @returns {ASTNode[]} Function nodes.
          * @private
          */
@@ -252,9 +252,8 @@ module.exports = {
 
         /**
          * Checks the position of given nodes.
-         *
-         * @param {ASTNode} inner - A node which is expected as inside.
-         * @param {ASTNode} outer - A node which is expected as outside.
+         * @param {ASTNode} inner A node which is expected as inside.
+         * @param {ASTNode} outer A node which is expected as outside.
          * @returns {boolean} `true` if the `inner` node exists in the `outer` node.
          * @private
          */
@@ -275,9 +274,8 @@ module.exports = {
          * - The reference is inside of a loop.
          * - The reference is inside of a function scope which is different from
          *   the declaration.
-         *
-         * @param {eslint-scope.Reference} ref - A reference to check.
-         * @param {ASTNode} prevRhsNode - The previous RHS node.
+         * @param {eslint-scope.Reference} ref A reference to check.
+         * @param {ASTNode} prevRhsNode The previous RHS node.
          * @returns {ASTNode|null} The RHS node or null.
          * @private
          */
@@ -310,9 +308,8 @@ module.exports = {
         /**
          * Checks whether a given function node is stored to somewhere or not.
          * If the function node is stored, the function can be used later.
-         *
-         * @param {ASTNode} funcNode - A function node to check.
-         * @param {ASTNode} rhsNode - The RHS node of the previous assignment.
+         * @param {ASTNode} funcNode A function node to check.
+         * @param {ASTNode} rhsNode The RHS node of the previous assignment.
          * @returns {boolean} `true` if under the following conditions:
          *      - the funcNode is assigned to a variable.
          *      - the funcNode is bound as an argument of a function call.
@@ -367,9 +364,8 @@ module.exports = {
          * - the function is bound as an argument of a function call.
          *
          * If a reference exists in a function which can be used later, the reference is read when the function is called.
-         *
-         * @param {ASTNode} id - An Identifier node to check.
-         * @param {ASTNode} rhsNode - The RHS node of the previous assignment.
+         * @param {ASTNode} id An Identifier node to check.
+         * @param {ASTNode} rhsNode The RHS node of the previous assignment.
          * @returns {boolean} `true` if the `id` node exists inside of a function node which can be used later.
          * @private
          */
@@ -385,9 +381,8 @@ module.exports = {
 
         /**
          * Checks whether a given reference is a read to update itself or not.
-         *
-         * @param {eslint-scope.Reference} ref - A reference to check.
-         * @param {ASTNode} rhsNode - The RHS node of the previous assignment.
+         * @param {eslint-scope.Reference} ref A reference to check.
+         * @param {ASTNode} rhsNode The RHS node of the previous assignment.
          * @returns {boolean} The reference is a read to update itself.
          * @private
          */
@@ -416,8 +411,7 @@ module.exports = {
 
         /**
          * Determine if an identifier is used either in for-in loops.
-         *
-         * @param {Reference} ref - The reference to check.
+         * @param {Reference} ref The reference to check.
          * @returns {boolean} whether reference is used in the for-in loops
          * @private
          */
@@ -453,7 +447,7 @@ module.exports = {
 
         /**
          * Determines if the variable is used.
-         * @param {Variable} variable - The variable to check.
+         * @param {Variable} variable The variable to check.
          * @returns {boolean} True if the variable is used
          * @private
          */
@@ -481,8 +475,7 @@ module.exports = {
 
         /**
          * Checks whether the given variable is after the last used parameter.
-         *
-         * @param {eslint-scope.Variable} variable - The variable to check.
+         * @param {eslint-scope.Variable} variable The variable to check.
          * @returns {boolean} `true` if the variable is defined after the last
          * used parameter.
          */
@@ -497,8 +490,8 @@ module.exports = {
 
         /**
          * Gets an array of variables without read references.
-         * @param {Scope} scope - an eslint-scope Scope object.
-         * @param {Variable[]} unusedVars - an array that saving result.
+         * @param {Scope} scope an eslint-scope Scope object.
+         * @param {Variable[]} unusedVars an array that saving result.
          * @returns {Variable[]} unused variables of the scope and descendant scopes.
          * @private
          */