-
Notifications
You must be signed in to change notification settings - Fork 45
Set postcss deps as peerDeps? #5
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
Comments
Aren't peerDeps deprecated in npm? I heard, they are going to remove it completely in the npm v3.0.0 |
well… no. They're not removed, they're changing behavior. In npm 2, peerDeps were auto-installed, and installs broke when this auto-install failed. In npm 3, peerDeps aren't auto-installed. It's up to the user to install the version they want. |
Well, I lack information about the topic, so I don't to know to what possible problems that solution can lead. Let me think a little about it. Currently, I want to fix test, so it will be possible to publish it. After that, I'll look on it closely. |
You should definitely use peerDeps, they work fine and they will for even better. |
I have to agree with you. Having postcss and its plugins as peerDeps is a good idea. It will be convenient both for the user and for maintainer (less updates). Documentation:
pr: #12 |
I published update, so you may try to use it :) |
It might be cleaner to have all the postcss deps as peerDeps? That way users of this module can set their own versions.
The text was updated successfully, but these errors were encountered: