-
Notifications
You must be signed in to change notification settings - Fork 62
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
Move postcss to peerDependencies #127
Comments
@ilfa Is this the proper way to handle this nowadays? Any docs for PostCSS specifically? Looking through a bunch of popular PostCSS plugins, they aren't doing this:
I do see that Autoprefixer does this: |
@MadLittleMods since the move to postcss 8, in the Plugin BoilerPlate Repo they are showing postcss as a peerDependency. edit: I found the migration guide, and it is now an official recommendation: |
Thanks for the context @ScottAwesome! @ilfa Feel free to create a PR 👍 |
Fixed in #129 and shipped in Thanks to @delucis for the contribution ❤️ |
It will be great not to download extra version of
postcss
.More details you can find in this article https://nodejs.org/en/blog/npm/peer-dependencies/.
I can create a PR if you are ok.
The text was updated successfully, but these errors were encountered: