We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I created a new draft PR that includes a new custom ESLint rule for that. #35448
Array
Symbol
Array.from
Symbol.for
Here is the result of make lint-js with new rule on the commit 4d7015f.
make lint-js
If we have a special reason for using the built-in object, we can add an inline comment such as eslint-disable node-core/prefer-primordials.
eslint-disable node-core/prefer-primordials
I'm not sure whether all codes in lib/ should be replaced with primordials.
lib/
What do you think?
The text was updated successfully, but these errors were encountered:
Fixed by cef1444.
Sorry, something went wrong.
Successfully merging a pull request may close this issue.
Problems
Proposal
I created a new draft PR that includes a new custom ESLint rule for that.
#35448
Array
orSymbol
Array.from
orSymbol.for
Here is the result of
make lint-js
with new rule on the commit 4d7015f.If we have a special reason for using the built-in object, we can add an inline comment such as
eslint-disable node-core/prefer-primordials
.Pros
Concerns
I'm not sure whether all codes in
lib/
should be replaced with primordials.What do you think?
The text was updated successfully, but these errors were encountered: