Code review comment for lp:~verterok/loggerhead/logging

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

I made some clean-up type changes in https://code.edge.launchpad.net/~mwhudson/loggerhead/logging.

I'm not sure that displaying "[(<class 'bzrlib.errors.NoSuchId'>, NoSuchId(), <traceback object at 0x1586680>)]" to the user really helps them very much. Can you maybe make this end of things a little nicer?

Other than that, it looks good!

« Back to merge proposal