Skip to content

fix: disallow extra properties in rule options#299

Merged
Pixel998 merged 1 commit into
eslint:mainfrom
andreww2012:fix/disallow-extra-properties-in-rule-options
Oct 13, 2025
Merged

fix: disallow extra properties in rule options#299
Pixel998 merged 1 commit into
eslint:mainfrom
andreww2012:fix/disallow-extra-properties-in-rule-options

Conversation

@andreww2012

Copy link
Copy Markdown
Contributor

Prerequisites checklist

What is the purpose of this pull request?

This PR simply disallows extra properties in rules' schemas which currently allow them.

What changes did you make? (Give an overview)

Related Issues

This PR is similar to #197, but this time disallows extra properties for the recently added selector-complexity rule.

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

@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage Oct 13, 2025
@eslint-github-bot eslint-github-bot Bot added the bug Something isn't working label Oct 13, 2025
@linux-foundation-easycla

linux-foundation-easycla Bot commented Oct 13, 2025

Copy link
Copy Markdown

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

  • ✅ login: andreww2012 / name: Andrew Kazakov (9c66227)

@Pixel998 Pixel998 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!

@Pixel998 Pixel998 added the accepted There is consensus among the team that this change meets the criteria for inclusion label Oct 13, 2025
@Pixel998 Pixel998 merged commit baf36e2 into eslint:main Oct 13, 2025
22 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Triage to Complete in Triage Oct 13, 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 bug Something isn't working

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

2 participants