Skip to content

Don't set log level for other packages #81

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

Merged
merged 1 commit into from
Mar 10, 2022

Conversation

Anton0
Copy link
Contributor

@Anton0 Anton0 commented Mar 9, 2022

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

change global configuration setting

  • What is the current behavior? (You can also link to an open issue here)

we are setting the log level for other packages boto, botocore and urllib3

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

features will still work but anyone who was using our lib to set boto log levels will need to do it themselves

@Anton0 Anton0 requested a review from a team as a code owner March 9, 2022 22:38
@Anton0 Anton0 force-pushed the do-not-set-log-levels-for-other-lib branch from a6cafd4 to 2c21a1f Compare March 9, 2022 23:50
@codecov
Copy link

codecov bot commented Mar 9, 2022

Codecov Report

Merging #81 (2c21a1f) into master (0ea560b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #81   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          438       434    -4     
=========================================
- Hits           438       434    -4     
Impacted Files Coverage Δ
staxapp/config.py 100.00% <ø> (ø)

@samdammers samdammers merged commit 708905e into master Mar 10, 2022
@samdammers samdammers deleted the do-not-set-log-levels-for-other-lib branch March 10, 2022 22:01
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