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

Two minor improvements - quiet variable and using stderr #80

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

Conversation

straxhaber
Copy link

Quiet Variable

The quiet variable, in particular, is important when being used as a library. Users might not want to see so much content. The 2FA OTP is the one thing that always needs to be shown. By default, I left the verbosity as-is, and just added a command-line flag to turn it off.

stderr

Right now, most regular output is going to stdout. Status updates intended for the user should go to stderr rather than stdout. stdout should be for actual data that was returned by an intended command, not these status update messages.

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