-
Notifications
You must be signed in to change notification settings - Fork 6
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
"Interesting" major bumps on peer deps #77
Comments
We solved this issue for now by switch from "peerDependencies": {
"some-dependency": "workspace:*"
}, to "peerDependencies": {
"some-dependency": "workspace:^"
}, but would've expected release-plan to instead check the |
@MarianBeuther showed the "fix" we did - I wanna dive into our reverse engineering of what we think was the thinking behind the decision making of Here is our "explanation" why we assume, this turns into a major version - and that has to do with automated updates. So, whenever a peer dep is bumped, that means automated tools such as renovate/dependabot wouldn't be able to automatically increase the version as it would also require them to also update the peer dep. That's why we think To precisely describe what we were after is actually: So We were left with the expectations Marian already posted.
Thanks |
FWIW, I was running into a similar issue recently with |
In our private monorepo where we are using
release-plan
, we started using it, when every package had the same version number.release-plan
made it so each package "ages" differently. While the majority of them isv5.*
I notied two packages are atv8
already - and nothing major happened to them.Looking at the explanations, these two packages have a peer dep to another package of the monorepo and that package is minor/patch-bumped when its dependencies receive a change. But each these bumps causes the beforementioned packages to major bump.
To my knowledge, that shouldn't cause a major bump, but a minor/patch respectively? Then this is a bug - or - my knowledge is wrong, then this is a question on what is the reason behind this =)
The text was updated successfully, but these errors were encountered: