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

v3 - Drop support for node 12, simplify recommended config in light of stylelint v15 deprecations #286

Merged
merged 4 commits into from
Feb 23, 2023

Conversation

BPScott
Copy link
Member

@BPScott BPScott commented Feb 23, 2023

Changes for v3 of the plugin.

  • Drops support for the long-EOL node 12.
  • Change the stylelint-prettier/recommended config to no longer try to include the stylelint-config-prettier config, as will not be needed as people move to stylelint 15 and stop using the deprecated stylistic rules.
  • Updates tests to account for changes in output in new stylelint v14 versions and changes to custom syntaxes in v15.
  • Update all devDependences

v2 of stylelint-prettier worked great when using Stylelint v15, however retaining test support for node 12, and stylelint v15 at the same time isn't worth the complication. It's much easier just to drop node 12 support.

@BPScott BPScott force-pushed the v3 branch 8 times, most recently from a401523 to 055401d Compare February 23, 2023 01:31
@BPScott BPScott changed the title WIP v3 - Drop support for node 12, simplify recommended config in light of stylelint v15 deprecations Feb 23, 2023
- Drop support for node 12
- Run tests using stylelint v15
- Update recommended config to only enable the prettier/prettier rule.
  It is now up to consumers to enable stylelint-config-prettier if they
  desire as stylelint v15 deprecates all the rules that
  stylelint-config-prettier disabled
- Tweak expected test output to account for error counts
- Tweak e2e test config to account for changes in css-in-js behaviour
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant