-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add level
option to setup_logging
#116
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #116 +/- ##
==========================================
+ Coverage 85.56% 85.94% +0.37%
==========================================
Files 8 8
Lines 568 569 +1
Branches 119 120 +1
==========================================
+ Hits 486 489 +3
+ Misses 67 66 -1
+ Partials 15 14 -1 ☔ View full report in Codecov by Sentry. |
a700e77
to
a1a5506
Compare
Hello @PicoCentauri! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-04-03 11:26:59 UTC |
Add `level` option to `setup_logging ` (#116)
Replace Boolean
debug
option insetup_logging
by more flexible integerlevel
parameter. This allows downstream packages to use this function in a better way.