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

Quiet Mode #181

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

Quiet Mode #181

wants to merge 2 commits into from

Conversation

ldv8434
Copy link

@ldv8434 ldv8434 commented May 6, 2022

Changes

  • Non-critical-error output is moved from stderr to stdout
    • FAIL actions were being logged at "Error" level, but do not halt patator execution, so they are also moved to stdout.
    • Anything logged as logging.CRITICAL level will continue to go to stderr to make future development easier.
  • A new flag (-q) will suppress non-critical-error output.
    • Only stdout is affected. All other forms of output are untouched.

Rationale

This is partially in response to #60 and partially for my company's own use in automating use of Patator.
The workaround suggested in the linked issue is to pipe stderr to /dev/null and use one of the other flags to write to an output file. This does solve the problem, but it also creates the issue of "hiding" critical errors.
This change keeps errors visible and makes working with command-line output easier by writing to the dedicated stdout stream.

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