Skip to content

feat: add no-invalid-named-grid-areas rule#169

Merged
snitin315 merged 4 commits into
eslint:mainfrom
xbinaryx:no-invalid-named-grid-areas
Jun 27, 2025
Merged

feat: add no-invalid-named-grid-areas rule#169
snitin315 merged 4 commits into
eslint:mainfrom
xbinaryx:no-invalid-named-grid-areas

Conversation

@xbinaryx

@xbinaryx xbinaryx commented Jun 9, 2025

Copy link
Copy Markdown
Contributor

Prerequisites checklist

What is the purpose of this pull request?

This PR adds a new rule no-invalid-named-grid-areas to enforce valid named grid area definitions in CSS Grid templates. The rule helps prevent common mistakes when using grid-template-areas and related properties by checking that named grid areas are properly defined according to the CSS Grid specification.

What changes did you make? (Give an overview)

Added the no-invalid-named-grid-areas rule, along with documentation and tests.

Related Issues

Fixes #158

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

@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage Jun 9, 2025
@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label Jun 10, 2025
@lumirlumir lumirlumir moved this from Needs Triage to Implementing in Triage Jun 10, 2025
@lumirlumir lumirlumir moved this from Implementing to Needs Triage in Triage Jun 10, 2025
nzakas
nzakas previously approved these changes Jun 10, 2025

@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! Would like another review before merging.

@nzakas nzakas moved this from Needs Triage to Second Review Needed in Triage Jun 10, 2025
Comment thread src/rules/no-invalid-named-grid-areas.js
Comment thread src/rules/no-invalid-named-grid-areas.js
Comment thread src/rules/no-invalid-named-grid-areas.js
Comment thread tests/rules/no-invalid-named-grid-areas.test.js
Comment thread tests/rules/no-invalid-named-grid-areas.test.js

@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. Would like @snitin315 to verify before merging.

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

@snitin315 snitin315 merged commit 162f6e5 into eslint:main Jun 27, 2025
22 checks passed
@github-project-automation github-project-automation Bot moved this from Second Review Needed to Complete in Triage Jun 27, 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 contributor pool feature

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

New Rule: no-invalid-named-grid-areas

4 participants