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

Make logging behave #12

Merged
merged 1 commit into from
Mar 14, 2015
Merged

Make logging behave #12

merged 1 commit into from
Mar 14, 2015

Conversation

wbolster
Copy link

This PR removes all custom logging magic, and changes the behaviour to
the standard way of logging for Python libraries.

In short, instead of magic global loggers and setting global
configuration (so that applications experience problems), all logging
calls are simply emitted on module-level loggers. It's up to the
application to route these messages somewhere useful by configuring
handlers and setting levels.

Fixes #11.

This PR removes all custom logging magic, and changes the behaviour to
the standard way of logging for Python libraries.

In short, instead of magic global loggers and setting global
configuration (so that applications experience problems), all logging
calls are simply emitted on module-level loggers. It's up to the
application to route these messages somewhere useful by configuring
handlers and setting levels.

Fixes #11.
@landscape-bot
Copy link

Code Health
Repository health decreased by 0.22% when pulling 88a7f69 on wbolster:make-logging-behave-properly into 64ea3bd on ssato:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.97%) to 94.42% when pulling 88a7f69 on wbolster:make-logging-behave-properly into 64ea3bd on ssato:master.

@ssato ssato merged commit db7284f into ssato:master Mar 14, 2015
@ssato
Copy link
Owner

ssato commented Mar 14, 2015

Thanks a lot for the fix. You're right. Merged as it is.

@wbolster
Copy link
Author

Great @ssato. Any chance you could publish a new release with this fix? Thanks!

@ssato
Copy link
Owner

ssato commented Mar 15, 2015

I've just released the new version that fix was included. Please let me know if you any issues w/ it.
And, again, thanks for the fix!

@wbolster
Copy link
Author

Awesome, thanks @ssato.

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.

anyconfig should not attach its own logging hanlers
5 participants