Skip to content

Commit

Permalink
tools: remove eslint rule no-let-in-for-declaration
Browse files Browse the repository at this point in the history
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]>
  • Loading branch information
gengjiawen authored and targos committed Mar 27, 2019
1 parent 13eb1d8 commit d80cd50
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 85 deletions.
1 change: 0 additions & 1 deletion lib/.eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down
45 changes: 0 additions & 45 deletions test/parallel/test-eslint-no-let-in-for-declaration.js

This file was deleted.

1 change: 0 additions & 1 deletion test/root.status
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
38 changes: 0 additions & 38 deletions tools/eslint-rules/no-let-in-for-declaration.js

This file was deleted.

0 comments on commit d80cd50

Please sign in to comment.