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

add additional verbosity options #697

Merged
merged 1 commit into from May 4, 2019
Merged

add additional verbosity options #697

merged 1 commit into from May 4, 2019

Conversation

grosser
Copy link
Owner

@grosser grosser commented May 4, 2019

replaces #694

--verbose currently displays both the command to be executed by each parallel
process before the tests run, and, on failure of a process, the command
needed to rerun the failing process.

We prefer to display only the latter.

This patch adds flags that can be used instead of --verbose to enable
either the per-process command, or the rerun command, or both.

--verbose-process-command will show the per-process commands, even if
--verbose is not set, or --quiet is set.

--verbose-rerun-command will show the rerun command, even if --verbose
is not set, or --quiet is set.

If --verbose is set, both types of command will be displayed, regardless
of whether the flags for those commands are set or not.

@alboyadjian

@grosser
Copy link
Owner Author

grosser commented May 4, 2019

@alboyadjian I bet quiet/verbose/verbose-flavors could use some rework too ... like quiet suppressing non-verbose output so that uses can use quiet and verbose together and we can remove the warning ... but good enough for now ...

@alboyadjian
Copy link

This will totally do what we need to do and the tests are cleaner. Thank you.

Thanks very much for creating (and maintaining) this tool, as well as the many others that I have used over the years.

@grosser grosser merged commit 18468eb into master May 4, 2019
@grosser grosser deleted the grosser/verbose branch May 4, 2019 21:02
@grosser
Copy link
Owner Author

grosser commented May 4, 2019

v2.29.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants