use latest version of pip in CI pipeline #2846
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Out of curiosity, do we know that pip isn't already initialized to the highest version by default when Python is set up? |
Yes, I know this to be true from experience. Recently GitHub actions started to show a warning when this happens for py3.10. It's also a common pattern that I use when building containers and CI pipelines but it hasn't been implemented here initially. It's worth adding imo. |
purificant
force-pushed
the
ci_use_latest_pip
branch
from
b6495f3
to
75d7412
Compare
|
Rebased the branch to use latest code from develop now that #2820 was merged. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Small tweak to CI.
1 Use latest version of pip to install dependencies. There can be some variation depending on which python version is used, this ensures the latest version.
2 Use
--no-cache-dirduring dependency installation, this avoid creating local pip cache which is redundant because it is discarded between runs and because we are caching the entire python environment with all installed dependencies.This PR should not conflict with #2820, however, it's probably a good idea to merge #2820 first.