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
postcss-nesting
: make edition 2024-02
the default
#1370
postcss-nesting
: make edition 2024-02
the default
#1370
Conversation
plugins/postcss-nesting/README.md
Outdated
## Options | ||
|
||
### edition | ||
|
||
The default behavior is to transpile CSS following an older version of the CSS nesting specification. | ||
The default behavior is to transpile CSS following the version of the CSS nesting specification as it existed during the last major version of this plugin. |
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.
I'm not sure this sentence is clear enough.
Is there more recent work to the spec? While I know we don't need to explain the specs I feel it doesn't hurt to be more explicit here.
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.
I was being vague on purpose but I see know that we need to be clearer on why this is vague.
I didn't want to have to change this sentence each time we ship a new edition or change the default.
Maybe better to explain why we have this mechanic instead.
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.
@Antonio-Laguna I've rewritten this section which more focus on what it is and when users should set this option. Hopefully an improvement :)
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.
This LGTM!
I left minor comments around the README for a bit of discussion but this is looking good! Great thing the tests were already separated and they weren't like _tape-default.mjs
, nice forward thinking!
…9ed' into postcss-nesting-make-2024-the-default--sympathetic-spadefoot-toad-3bea5e0b0d
086320b
into
v10--gregarious-seahorse-86cb98a9ed
2024-02
the default inpostcss-nesting