-
-
Notifications
You must be signed in to change notification settings - Fork 71
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
plugins-stylelint/no-at-nest-rule
#997
plugins-stylelint/no-at-nest-rule
#997
Conversation
…c-zebra-shark-a8b34dabd3
…at-nest-rule--empathetic-zebra-shark-a8b34dabd3
…at-nest-rule--empathetic-zebra-shark-a8b34dabd3
@@ -147,6 +147,11 @@ export const osiApprovedWithCCO = [ | |||
]; | |||
|
|||
const licenseExceptions = [ | |||
{ | |||
// because ... irony? 🤷 |
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.
😂
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.
@romainmenke I agree completely!
I like how nice the tests are.
Does this need any special handling from the release script?
I ran the dry run thingy and that indicated that it would just work. |
A Stylelint plugin to change nested css with
@nest
to css without.Effectively migrating to the latest specification.
I am not going to bother supporting the non-standard nesting that Chrome and Safari shipped.
i.e. I am not going to wrap stuff with
:is()
.The most important thing for us is that our users stop using
@nest
.@Antonio-Laguna We discussed at some point that we would also create a monorepo for the Stylelint plugins under
@csstools
. I think this is still worthwhile but separate from this change.If any Stylelint plugins are created in this repo they should be tightly coupled to a PostCSS plugin.
Let me know if you disagree :)