Skip to content
New issue

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

tools: fix bugs in prefer-primordials linter rule #42010

Merged
merged 2 commits into from Feb 18, 2022

Conversation

@aduh95
Copy link
Contributor

@aduh95 aduh95 commented Feb 16, 2022

The ESLint rule would repport false positive if code is using an
identifier that happens to have the same name as a primordials member.

The ESLint rule would repport false positive if code is using an
identifier that happens to have the same name as a primordials member.
Leko
Leko approved these changes Feb 16, 2022
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Feb 16, 2022

@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Feb 17, 2022

Co-authored-by: Rich Trott <rtrott@gmail.com>
Trott
Trott approved these changes Feb 18, 2022
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Feb 18, 2022

@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Feb 18, 2022

@nodejs-github-bot nodejs-github-bot merged commit 6700b5c into nodejs:master Feb 18, 2022
55 checks passed
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Feb 18, 2022

Landed in 6700b5c

@aduh95 aduh95 deleted the fix-prefer-primordials branch Feb 18, 2022
bengl added a commit to bengl/node that referenced this issue Feb 21, 2022
The ESLint rule would repport false positive if code is using an
identifier that happens to have the same name as a primordials member.

PR-URL: nodejs#42010
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
bengl added a commit to bengl/node that referenced this issue Feb 21, 2022
The ESLint rule would repport false positive if code is using an
identifier that happens to have the same name as a primordials member.

PR-URL: nodejs#42010
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
bengl added a commit that referenced this issue Feb 21, 2022
The ESLint rule would repport false positive if code is using an
identifier that happens to have the same name as a primordials member.

PR-URL: #42010
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@bengl bengl mentioned this pull request Feb 21, 2022
bengl added a commit that referenced this issue Feb 21, 2022
The ESLint rule would repport false positive if code is using an
identifier that happens to have the same name as a primordials member.

PR-URL: #42010
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
bengl added a commit that referenced this issue Feb 22, 2022
The ESLint rule would repport false positive if code is using an
identifier that happens to have the same name as a primordials member.

PR-URL: #42010
Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants