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
Restore TOX_SKIP_ENV filtering #2707
Merged
Merged
Commits on Dec 15, 2022
-
Restore TOX_SKIP_ENV filtering
This is still documented in docs/config.rst. The only missing thing is the reporting at verbosity level 2. Fixes tox-dev#2698.
Configuration menu - View commit details
-
Copy full SHA for 8a28e88 - Browse repository at this point
Copy the full SHA 8a28e88View commit details -
Configuration menu - View commit details
-
Copy full SHA for a6887fa - Browse repository at this point
Copy the full SHA a6887faView commit details -
Log about skipped environments
Logging this at DEBUG level (tox -vv) felt wrong to me, so I went with INFO (tox -v).
Configuration menu - View commit details
-
Copy full SHA for fe2d258 - Browse repository at this point
Copy the full SHA fe2d258View commit details -
Configuration menu - View commit details
-
Copy full SHA for c101da7 - Browse repository at this point
Copy the full SHA c101da7View commit details -
Change the log level to WARNING to match the documentation
tox --help says verbosity level 2 is WARNING.
Configuration menu - View commit details
-
Copy full SHA for 89572aa - Browse repository at this point
Copy the full SHA 89572aaView commit details -
As far as I can tell, during runtime there's only one EnvSelector instance, so there's no danger of repeatedly warning about the same skipped environment when .iter() gets called several times. Fixes failing unit tests. (Oops. I only ran each test in isolation, because the entire test suite is a bit slow.)
Configuration menu - View commit details
-
Copy full SHA for c13c9db - Browse repository at this point
Copy the full SHA c13c9dbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 924e32c - Browse repository at this point
Copy the full SHA 924e32cView commit details -
Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
Configuration menu - View commit details
-
Copy full SHA for ad3e3ed - Browse repository at this point
Copy the full SHA ad3e3edView commit details -
Configuration menu - View commit details
-
Copy full SHA for 449d884 - Browse repository at this point
Copy the full SHA 449d884View commit details
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.