Skip to content

Conversation

@Tanujkanti4441
Copy link
Contributor

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:

What changes did you make? (Give an overview)

Updated the error location of no-dupe-class-members rule to the identifier instead of the whole expression so that it will only highlight the place where we need to change the code to fix the error.

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

Similar to PR #20258

@Tanujkanti4441 Tanujkanti4441 requested a review from a team as a code owner October 30, 2025 05:11
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Oct 30, 2025
@eslint-github-bot eslint-github-bot bot added the feature This change adds a new feature to ESLint label Oct 30, 2025
@github-actions github-actions bot added the rule Relates to ESLint's core rules label Oct 30, 2025
@netlify
Copy link

netlify bot commented Oct 30, 2025

Deploy Preview for docs-eslint ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/projects/docs-eslint/deploys/69034900cced1d20f6b784a0
😎 Deploy Preview https://deploy-preview-20259--docs-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label Oct 30, 2025
@lumirlumir lumirlumir moved this from Needs Triage to Triaging in Triage Oct 30, 2025
Copy link
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Would like another review before merging.

@lumirlumir lumirlumir moved this from Triaging to Second Review Needed in Triage Oct 30, 2025
Copy link
Member

@mdjermanovic mdjermanovic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! Leaving open for one more day if anyone else would like to review it.

@mdjermanovic mdjermanovic moved this from Second Review Needed to Merge Candidates in Triage Oct 30, 2025
@mdjermanovic mdjermanovic changed the title feat: update error loc to id in no-dupe-class-members feat: update error loc to key in no-dupe-class-members Oct 31, 2025
@mdjermanovic mdjermanovic merged commit cc57d87 into eslint:main Oct 31, 2025
32 checks passed
@github-project-automation github-project-automation bot moved this from Merge Candidates to Complete in Triage Oct 31, 2025
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 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.

3 participants