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.

Changes between revision 7 and 8

orig
1
2
3
4
5
6
7
8

Commits

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
fixing indent errors
3d387ef246879fc77562bd1e538f4562cb16f2d5 anahita-m
fixing formatting issue with imports
fa90587358957685d770fc673b75f939e8216a5c 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...