-
-
Notifications
You must be signed in to change notification settings - Fork 526
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
option to disable parallel spinner for CI envs #1184
Comments
N.B. Place in code: https://github.com/tox-dev/tox/pull/1102/files#diff-fa162e4b845a9c35e2b680c199480dc6R634
|
@gaborbernat Can you pleaseedi the bug description to make it easier to read? A link to another issue does not help much. PS. I cannot edit it myself. |
@ssbarnea addressed |
Thanks. Indeed the progress-bar (or keep-alive) may find useful to indicate to ci or humans that the task is not stuck. At the same time it could only be seen as a source of spam for both ci (where it can create undesired extra logs) or for users, where it can set the modified flags in the terminal windows, without real reasons (I seen this with iterm tabs too). Being able to tune it is highly desired. |
Maybe something similar to unittest's dot output per test case? However, we'd have to iterate on something else. e.g., maybe output a dot every second or so when a subprocess has written to stdout? This should reduce spam, while still enabling keep-alive. |
In some cases (CIs mostly) it makes sense to not pollute the output with the progress bar, let's allow somehow to disable it in such rare cases 👍 #1142 An environment variable will suffice.
The text was updated successfully, but these errors were encountered: