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.

Sign up for GitHub

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

Jump to bottom

- Put first logging instance after basicConfig call #60

Merged

wdecoster merged 1 commit into wdecoster:master from alvanuffelen:log_fix

Sep 6, 2022

Contributor

alvanuffelen commented Sep 6, 2022

closes #50

The Logging docs state:

The call to basicConfig() should come before any calls to debug(), info(), etc. Otherwise, those functions will call basicConfig() for you with the default options. As it’s intended as a one-off simple configuration facility, only the first call will actually do anything: subsequent calls are effectively no-ops.


          - Put first logging instance after basicConfig call

19712b7

wdecoster approved these changes

View reviewed changes

wdecoster merged commit 5a8a3ee into wdecoster:master

Owner

wdecoster commented Sep 6, 2022

Aha, much appreciated!

alvanuffelen deleted the log_fix branch

September 6, 2022 17:15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet