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

Allow testenv to be "non-voting" (optional) #174

Closed
pytoxbot opened this issue Sep 17, 2016 · 2 comments
Closed

Allow testenv to be "non-voting" (optional) #174

pytoxbot opened this issue Sep 17, 2016 · 2 comments

Comments

@pytoxbot
Copy link

@pytoxbot pytoxbot commented Sep 17, 2016

I would like to see an option that tells tox to not fail for testenvs / jobs that are considered optional.

For example, if I have a section [testenv:pep8] I don't want tox to fail if I mark it as non-voting. So if a testenv returns a non-zero status (failure) then tox would still exit with a zero status but display a warning that a non-voting test failed.

#!ini

[tox]
skipsdist = True
envlist = py26, py27, pep8

[testenv]
commands =
        pip install -e .
        pip install -r test-requirements.txt
        py.test {posargs}

[testenv:pep8]
voting = False
commands =
        pip install pep8
        pep8 --ignore=E501 myproject/

If this is considered a good idea, I'd like to try implementing it.

@pytoxbot
Copy link
Author

@pytoxbot pytoxbot commented Sep 17, 2016

Original comment by @hpk42

fix issue174: add new ignore_outcome testenv attribute which
can be set to True in which case it will produce a warning instead
of an error on a failed testenv command outcome.
Thanks Rebecka Gulliksson for the PR.

→ <<cset 618ba985f958>>

@pytoxbot
Copy link
Author

@pytoxbot pytoxbot commented Sep 17, 2016

Original comment by @msabramo

Yes, this would be useful. I might also use this to monitor Python 3 compatibility without strictly requiring it.

@pytoxbot pytoxbot closed this Sep 17, 2016
@tox-dev tox-dev locked and limited conversation to collaborators Jan 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant