tools: remove eslint rule no-let-in-for-declaration

PR-URL: #26715 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Mar 27, 2019 · d80cd50 · d80cd50
1 parent 13eb1d8
commit d80cd50
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 85 deletions.
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -17,7 +17,6 @@ rules:
       message: "Use an error exported by the internal/errors module."
   # Custom rules in tools/eslint-rules
   node-core/require-globals: error
-  node-core/no-let-in-for-declaration: error
   node-core/lowercase-name-for-primitive: error
   node-core/non-ascii-character: error
 globals:

diff --git a/test/parallel/test-eslint-no-let-in-for-declaration.js b/test/parallel/test-eslint-no-let-in-for-declaration.js
diff --git a/test/root.status b/test/root.status
@@ -37,7 +37,6 @@ parallel/test-eslint-duplicate-requires: SLOW
 parallel/test-eslint-eslint-check: SLOW
 parallel/test-eslint-inspector-check: SLOW
 parallel/test-eslint-lowercase-name-for-primitive: SLOW
-parallel/test-eslint-no-let-in-for-declaration: SLOW
 parallel/test-eslint-no-unescaped-regexp-dot: SLOW
 parallel/test-eslint-number-isnan: SLOW
 parallel/test-eslint-prefer-assert-iferror: SLOW

diff --git a/tools/eslint-rules/no-let-in-for-declaration.js b/tools/eslint-rules/no-let-in-for-declaration.js