Skip to content

Commit

Permalink
tools: remove custom buffer-constructor lint rule
Browse files Browse the repository at this point in the history
This rule is no longer needed. ESLint's built in
no-buffer-constructor rule is enabled instead.

PR-URL: #25261
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
  • Loading branch information
cjihrig authored and targos committed Jan 1, 2019
1 parent ee43540 commit fee8a11
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 49 deletions.
1 change: 0 additions & 1 deletion lib/.eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,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/buffer-constructor: error
node-core/no-let-in-for-declaration: error
node-core/lowercase-name-for-primitive: error
node-core/non-ascii-character: error
Expand Down
27 changes: 0 additions & 27 deletions test/parallel/test-eslint-buffer-constructor.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 @@ -31,7 +31,6 @@ parallel/test-domain-with-abort-on-uncaught-exception: SLOW
parallel/test-env-var-no-warnings: SLOW
parallel/test-error-reporting: SLOW
parallel/test-eslint-alphabetize-errors: SLOW
parallel/test-eslint-buffer-constructor: SLOW
parallel/test-eslint-crypto-check: SLOW
parallel/test-eslint-documented-errors: SLOW
parallel/test-eslint-duplicate-requires: SLOW
Expand Down
20 changes: 0 additions & 20 deletions tools/eslint-rules/buffer-constructor.js

This file was deleted.

0 comments on commit fee8a11

Please sign in to comment.