Merge lp://staging/~spiv/loggerhead/logging-trivial into lp://staging/loggerhead

Proposed by Andrew Bennetts
Status: Merged
Approved by: Michael Hudson-Doyle
Approved revision: 428
Merged at revision: 428
Proposed branch: lp://staging/~spiv/loggerhead/logging-trivial
Merge into: lp://staging/loggerhead
Diff against target: 25 lines (+0/-4)
2 files modified
loggerhead/controllers/diff_ui.py (+0/-2)
loggerhead/controllers/inventory_ui.py (+0/-2)
To merge this branch: bzr merge lp://staging/~spiv/loggerhead/logging-trivial
Reviewer Review Type Date Requested Status
Michael Hudson-Doyle Approve
Review via email: mp+29710@code.staging.launchpad.net

Description of the change

Get rid of some unused 'log' globals.

(A 'loggerhead.controllers' logger is potentially still created by the @lsprof decorator in loggerhead.utils, but I'm not too bothered about debug-only code...)

To post a comment you must log in.
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Very deep, I like it. Can you merge this yourself?

review: Approve
Revision history for this message
Andrew Bennetts (spiv) wrote :

Michael Hudson wrote:
> Review: Approve
> Very deep, I like it. Can you merge this yourself?

Apparently I can!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches