-
Notifications
You must be signed in to change notification settings - Fork 5
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
[ENH] Use logging and let the user reduce the verbosity #162
Comments
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days. |
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days. |
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days.
|
Is there an existing issue for this?
New feature
Right now we tell the user good news with a print statement, e.g:
bagel-cli/bagel/cli.py
Lines 70 to 74 in 0ed0e4d
which means they can't turn that off if they'd prefer (e.g. when they run this thing a couple hundred times in a row).
We should instead use a logger (https://docs.python.org/3/library/logging.html).
This has other nice benefits such as:
In short: let's do this.
Unclear documentation
No response
The text was updated successfully, but these errors were encountered: