Code review comment for lp:~parthm/bzr/538868-message-for-heavy-checkout

Revision history for this message
Martin Pool (mbp) wrote :

Thanks, this is a very nice bug to fix.

I would prefer the message came out through trace or the ui factory
than directly to self.outf, because that will make it easier to
refactor out of the cmd implementation, and it's more likely to
automatically respect --quiet. You might then be able to test more
cleanly through TestUIFactory.

« Back to merge proposal