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

parallel bootstrap #10

Closed
viliusl opened this issue Jan 23, 2017 · 3 comments
Closed

parallel bootstrap #10

viliusl opened this issue Jan 23, 2017 · 3 comments

Comments

@viliusl
Copy link
Collaborator

viliusl commented Jan 23, 2017

This one is tricky, as you have to bootstrap in respect to dependency graph, but oh well.

Working code for inspiration is: https://github.com/laurynasl-wix/wnpm/commit/444cac4d7d8542fcae5a22c7bd94a940f1ffd845 + following commits for fixes.

Execution should just be properly tdd'ed of course:)

@viliusl
Copy link
Collaborator Author

viliusl commented Feb 25, 2017

@itaipelles - yarnpkg/yarn#683 - ouch:/

In any case afaik there is only a question when it will be resolved and not if, so let's proceed. Also given engine is configurable we can see if in parallel scenario npm works better or do some other optimizations here.

@laurynaslubys
Copy link
Contributor

Btw, I think this only affects the cases where your yarn install puts stuff into the global cache, so if you're bootstrapping after a previous one, and yarn doesn't have to fetch identical dependencies from the registry, it doesn't blow up.

@viliusl
Copy link
Collaborator Author

viliusl commented Mar 18, 2017

it works with npm now

@viliusl viliusl closed this as completed Mar 18, 2017
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

No branches or pull requests

2 participants