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

Suppress subcommand output by default #26

Merged
merged 1 commit into from
Oct 5, 2024

Conversation

AmaranthineCodices
Copy link
Collaborator

For large or busy repositories, stack-pr commands can generate hundreds
of lines of output from Git subcommands being executed. Most of the time
these don't matter; the user doesn't care about them. This PR suppresses
them by default by redirecting stdout and stderr to
subprocess.DEVNULL. If this output is desired in a specific case, the
command-line flag --verbose (-V) can be used to avoid this
redirection.

For large or busy repositories, stack-pr commands can generate hundreds
of lines of output from Git subcommands being executed. Most of the time
these don't matter; the user doesn't care about them. This PR suppresses
them by default by redirecting stdout and stderr to
`subprocess.DEVNULL`. If this output is desired in a specific case, the
command-line flag `--verbose` (`-V`) can be used to avoid this
redirection.
Copy link
Collaborator

@ZolotukhinM ZolotukhinM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ZolotukhinM ZolotukhinM merged commit a2af242 into main Oct 5, 2024
2 checks passed
@ZolotukhinM ZolotukhinM deleted the lbrown/quieter-by-default branch October 5, 2024 22:55
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.

2 participants