Replacing non-named usages to use names logging in files in reviewboard.notifications

Review Request #11396 — Created Jan. 22, 2021 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

This commit is replacing non-named usages in reviewboard/notifications to use
named logging instead. The purpose of this change is to automatically include
more information about the module where the logging message came from so that
the messages can be of more use when diagnosing problems from logs.

I ran the unit tests under reviewboard/notifications/tests by running
./tests/runtests.py reviewboard.notifications.tests and confirmed that all
162 tests ran without fail.

Diff Revision 7

This is not the most recent revision of the diff. The latest diff is revision 8. See what's changed.

orig
1
2
3
4
5
6
7
8

Commits

First Last Summary ID Author
Replacing non-named usages to use names logging in files in reviewboard.notif...
This commit is replacing non-named usages in reviewboard/notifications to use named logging instead. The purpose of this change is to automatically include more information about the module where the logging message came from so that the messages can be of more use when diagnosing problems from logs. I ran the unit tests under reviewboard/notifications/tests by running ./tests runtests.py reviewboard.notifications.tests and confirmed that all 162 tests ran without fail
613df8f5030d136b519bae0de9e9fc812a2533f6 anahita-m
reviewboard/notifications/webhooks.py
reviewboard/notifications/email/message.py
reviewboard/notifications/email/utils.py
reviewboard/notifications/tests/test_email_sending.py
reviewboard/notifications/tests/test_webhooks.py
Loading...