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
Add media-query-no-invalid
specific problem messages
#7462
Add media-query-no-invalid
specific problem messages
#7462
Conversation
🦋 Changeset detectedLatest commit: 4cc80c2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@romainmenke Thanks for the pull request!
As far as I know, there is no such case. But I think what this PR try is not a problem. 👌🏼
I prefer
I think so, too. Your try looks good to me. 👍🏼 |
…pe-71a22d3c66' of https://github.com/stylelint/stylelint into media-query-no-invalid--specific-messages--witty-antelope-71a22d3c66
Thank you for the feedback @Mouvedia, @ybiquitous 🙇 |
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.
Thank you. LGTM 👍🏼
Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
Closes #7401
I've added a second argument for the
rejected
message.hint
a good name for the second argument? (maybereason
, ...)I am also unsure what the best wording is.
I don't think most people are familiar with the wording of the specification, e.g.
range context
,plain feature
, ...So I've opted to label everything as a query :
range query
,boolean query
.