Add named logging in the extensions module.

Review Request #11398 — Created Jan. 22, 2021 and submitted

mfliu
Review Board
release-4.0.x
4904
reviewboard, students

The old global logging commands such as logging.error and
logging.info do not insert information about where the log message
came from. By instantiating a module-level logger object, log messages
will now include information about the source module.

Ran all reviewboard.extensions.tests unit tests.

Summary
Add named logging in the extensions module.
Description From Last Updated

Can you add the bug number to the "bugs" field?

daviddavid

Super tiny nit: please add a period at the end of your testing done.

daviddavid

Can you fix up indentation here? logger is one character shorter than logging.

daviddavid

Same here.

daviddavid

And here.

daviddavid
mfliu
amohapatra
  1. Ship It!
  2. 
      
david
  1. 
      
  2. Can you add the bug number to the "bugs" field?

  3. Super tiny nit: please add a period at the end of your testing done.

  4. 
      
mfliu
david
  1. 
      
  2. Can you fix up indentation here? logger is one character shorter than logging.

  3. 
      
mfliu
david
  1. Ship It!
  2. 
      
mfliu
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.0.x (1395d3b)
Loading...