Skip to content

fix: skip descriptor validation for nestable at-rules#362

Merged
nzakas merged 2 commits into
eslint:mainfrom
xbinaryx:nestable-at-rules-descriptor-validation
Jan 15, 2026
Merged

fix: skip descriptor validation for nestable at-rules#362
nzakas merged 2 commits into
eslint:mainfrom
xbinaryx:nestable-at-rules-descriptor-validation

Conversation

@xbinaryx

Copy link
Copy Markdown
Contributor

Prerequisites checklist

What is the purpose of this pull request?

The no-invalid-at-rules rule was incorrectly treating properties inside nestable at-rules (like @media, @supports, @layer, etc.) as descriptors.

What changes did you make? (Give an overview)

Added a check to skip descriptor validation entirely when a declaration appears inside a nestable at-rule, since these at-rules don't have descriptors and should allow properties directly.

Related Issues

Closes #255

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

@Tanujkanti4441 Tanujkanti4441 added the accepted There is consensus among the team that this change meets the criteria for inclusion label Jan 15, 2026
@Tanujkanti4441 Tanujkanti4441 moved this from Needs Triage to Implementing in Triage Jan 15, 2026

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

Would like another member to review.

@Tanujkanti4441 Tanujkanti4441 moved this from Implementing to Second Review Needed in Triage Jan 15, 2026

@nzakas nzakas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks!

@nzakas nzakas merged commit 50ff5c1 into eslint:main Jan 15, 2026
22 checks passed
@github-project-automation github-project-automation Bot moved this from Second Review Needed to Complete in Triage Jan 15, 2026
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 contributor pool

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

Bug: Nested media queries report when they directly use a descriptor

3 participants