Launchpad itself

Code review comment for lp:~thumper/launchpad/bugjam-619555-request-builds-logging

  1. bugjam-619555-request-builds-logging
  2. Merge into devel
Revision history for this message
Benji York (benji) wrote :

Nice, straight-forward branch.

The only thing I can suggest is that the conditional in TestLogger.log() is unnecessary. If args is an empty tuple, then "msg %= args" will leave msg as it was.

review: Approve (code*)

« Back to merge proposal