-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Release 1.1.0a3 #2648
Release 1.1.0a3 #2648
Conversation
46210cc
to
68c2cf7
Compare
68c2cf7
to
a50d87d
Compare
Do we need to do anything to enable #2595 or is it enabled by default? |
@aviramha It's enabled by default, yes. However, it's possible that on some CI systems the parallelization is not activated. |
What do you mean by that? What kind of parallelization needs to be enabled? |
@aviramha For now, there is nothing you can do to force the parallelization on some CI systems. Note that I assumed that you were not seeing speed improvements in your CI because that's the only case where the parallelization might not be enabled, however if it's not the case I would like to know to track down the issue. As a side note, the relevant code snippet that explains why it's not activated on some CI systems is here: https://github.com/python-poetry/poetry/blob/develop/poetry/installation/executor.py#L42 However, it was here more as a safety net in the first iterations on the feature and I think it can be removed now that it has been thoroughly tested. |
You're correct the improvement isn't seen on bitbucket pipelines. |
@sdispater I took the liberty of removing the check and sending a PR. I really want to speed up my CI :) |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Added
Fixed