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

Should changed packages include cross-depencies? #27

Closed
jamiebuilds opened this issue Jan 25, 2016 · 4 comments
Closed

Should changed packages include cross-depencies? #27

jamiebuilds opened this issue Jan 25, 2016 · 4 comments

Comments

@jamiebuilds
Copy link
Contributor

Right now when publishing a new version we:

  1. Find all the changed packages by checking their diffs. ref
  2. Cross check these "changed packages" for cross-deps and update them in package.json. ref
  3. However, we don't update cross-dependencies outside of the original changed packages. (ref1, ref2)

I feel like this will cause issues with major versions. Because we're relying on npm's ^x.x.x version matching for minor and patch versions outside of the changed packages.

@hzoo
Copy link
Contributor

hzoo commented Jan 25, 2016

Yeah I think you're right on that.
Could we do that only if it's a major version change? Thats kinda what FORCE_VERSION does (but it would update all packages).

@jamiebuilds
Copy link
Contributor Author

Only doing it on major versions sounds like the best solution. The question is if we recursively look up changed packages or bump them all.

I'm concerned about how this would interact with #20

@jamiebuilds
Copy link
Contributor Author

Fixed in 2.0.0-beta.4

@lock
Copy link

lock bot commented Dec 28, 2018

This thread has been automatically locked because there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants