Skip to content

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