New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix false negatives for grid shorthands in named-grid-areas-no-invalid #5514
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jeddy3 Thank you for creating the PR. I've left some refactoring suggestions, but it looks almost fine! 😄
Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
Thanks for the review and suggestions. I've committed them both. |
My suggestion code seems to be something wrong. 👀 |
By the change of indentation (80f5ab3), it is necessary to change the expectation of diff --git a/lib/rules/named-grid-areas-no-invalid/__tests__/index.js b/lib/rules/named-grid-areas-no-invalid/__tests__/index.js
index e49bc06d..4855c7fd 100644
--- a/lib/rules/named-grid-areas-no-invalid/__tests__/index.js
+++ b/lib/rules/named-grid-areas-no-invalid/__tests__/index.js
@@ -117,7 +117,7 @@ testRule({
}`,
message: messages.expectedToken(),
line: 2,
- column: 4,
+ column: 2,
},
{
code: 'a { grid-template-areas: "" "" ""; }', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jeddy3 Sorry for the inconvenience. LGTM! 👍🏼
No worries at all! I appreciate the review 😄 |
|
Ref: #5513 (review)
grid
andgrid-template
.