Skip to content

feat: add self to no-implied-eval rule - #20465

Closed
bgk614 wants to merge 1 commit into
eslint:mainfrom
bgk614:feat/no-implied-eval-add-self
Closed

bgk614 wants to merge 1 commit into
eslint:mainfrom
bgk614:feat/no-implied-eval-add-self

Conversation

@bgk614

@bgk614 bgk614 commented Jan 28, 2026

Copy link
Copy Markdown

Prerequisites checklist

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[x] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

Fixes #19950

What changes did you make? (Give an overview)

Added self to the list of global object candidates in the no-implied-eval rule.

Is there anything you'd like reviewers to focus on?

`self` is an alias for `window` and can be used for implied eval

Fixes eslint#19950
@bgk614
bgk614 requested a review from a team as a code owner January 28, 2026 07:45
@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage Jan 28, 2026
@eslint-github-bot eslint-github-bot Bot added the feature This change adds a new feature to ESLint label Jan 28, 2026
@netlify

netlify Bot commented Jan 28, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for docs-eslint canceled.

Name Link
🔨 Latest commit 202f5b5
🔍 Latest deploy log https://app.netlify.com/projects/docs-eslint/deploys/6979be952b59b20008ca406a

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jan 28, 2026 •

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: bgk614 / name: Bogyeong Kim (202f5b5)

@github-actions github-actions Bot added the rule Relates to ESLint's core rules label Jan 28, 2026

@snitin315 snitin315 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

Leaving it open for others to review.

@snitin315 snitin315 added accepted There is consensus among the team that this change meets the criteria for inclusion contributor pool labels Jan 28, 2026
@snitin315 snitin315 moved this from Needs Triage to Second Review Needed in Triage Jan 28, 2026
@sethamus

Copy link
Copy Markdown
Contributor

#19950 is assigned to me.

@bgk614

bgk614 commented Jan 28, 2026

Copy link
Copy Markdown
Author

#19950 is assigned to me.

@sethamus

Sorry, I didn't check the assignee before starting work on this.

Would you like to continue working on it? I'm happy to close this PR if you'd prefer to complete it yourself.

@sethamus

Copy link
Copy Markdown
Contributor

@bgk614 Thanks for checking! I’ve already opened #20468.

@bgk614

bgk614 commented Jan 28, 2026

Copy link
Copy Markdown
Author

@sethamus
Got it, thanks for letting me know! I’ll go ahead and close this PR then.

@bgk614 bgk614 closed this Jan 28, 2026
@github-project-automation github-project-automation Bot moved this from Second Review Needed to Complete in Triage Jan 28, 2026
@bgk614
bgk614 deleted the feat/no-implied-eval-add-self branch July 12, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion contributor pool feature This change adds a new feature to ESLint rule Relates to ESLint's core rules

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

Rule Change: Add self to global objects checked by no-implied-eval

3 participants