-
Notifications
You must be signed in to change notification settings - Fork 2
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
PIP_DISABLE_PIP_VERSION_CHECK=1 #19
Comments
Hmm, I'm not sure I want to disable the check by default. If you want to disable the warning, you can always set that environment variable in tox.ini with setenv/passenv as needed.(if pip environment vars aren't working with tox-pip-version, that's a big bug) Although, don't feel too strongly about it. I wouldn't be opposed to a PR that added it. |
The reason I thought it would be useful to support Another one worth considering support for is Since pip 10, While users can set these env vars themselves, that is beyond many people's ability / capacity / interest levels, as it requires reading the tox & pip docs, quite carefully to realise that pip uses envvars, and they can be passed through tox to its internal pip. They may even decide that some configuration is unsupportable because there was no way to switch off the behaviour they saw. tox-pip-version having switches with sensible defaults helps those who dont https://github.com/asottile-archive/tox-virtualenv-no-download/blob/master/tox_virtualenv_no_download.py is effectively a plugin which only sets one envvar which could have been set by the user. Arguably its existence resulted in tox 3.10 changing its default behaviour, or at least was a catalyst. As it sounds like pip might also be making this plugin redundant, I'll hold off providing a PR. I'm just getting ideas written down somewhere relevant :-) |
If the user has set a desired pip version, especially if a precise pin, there is no need for pip to do its version check.
PIP_DISABLE_PIP_VERSION_CHECK=1
disables that.Any reason not to add this?
The text was updated successfully, but these errors were encountered: