Skip to content
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

Conversation

romainmenke
Copy link
Member

@romainmenke romainmenke commented Apr 20, 2024

  • make 2024-02 the default in postcss-nesting
  • update test expect files everywhere
  • update the playground as this no longer needs this extra flag

@romainmenke romainmenke marked this pull request as ready for review April 20, 2024 10:40
@romainmenke romainmenke mentioned this pull request Apr 20, 2024
4 tasks
## 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.
Copy link
Member

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.

Copy link
Member Author

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.

Copy link
Member Author

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 :)

https://github.com/csstools/postcss-plugins/blob/postcss-nesting-make-2024-the-default--sympathetic-spadefoot-toad-3bea5e0b0d/plugins/postcss-nesting/README.md#edition

Copy link
Member

@Antonio-Laguna Antonio-Laguna left a 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
@romainmenke romainmenke merged commit 086320b into v10--gregarious-seahorse-86cb98a9ed May 5, 2024
6 checks passed
@romainmenke romainmenke deleted the postcss-nesting-make-2024-the-default--sympathetic-spadefoot-toad-3bea5e0b0d branch May 5, 2024 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants