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 support for explicit exit status, suppressing color output #120

Open
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

bww
Copy link

@bww bww commented Apr 3, 2019

Report error exit status

Adds support for supplying an explicit exit status that will be used if any of the managed processes exit with non-zero status. This is useful to report the fact that a process crashed to whatever process invoked forego.

This functionality is enabled by supplying the newly-defined -s flag with the desired exit status.

Suppress color

Colored output can optionally be suppressed by providing the -n flag. This can be useful in situations where ANSI escape codes are not correctly interpreted and result in garbage being printed to stdout.

Misc

Cleaned up a couple tests that used format specifiers in the non-formatting t.Fatal method.

@yb66 yb66 mentioned this pull request Sep 21, 2019
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.

1 participant