Code review comment for lp:~toonn/bzr/log-bzr-version

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

Thanks for the patch! It looks good. My only comment on it is: don't add a blank line after the trace.mutter, we generally don't use much vertical whitespace within functions. The typo fixes look good too.

Have you already executed the Canonical contribution agreement? If not, could you please do so according to <http://www.canonical.com/contributors> — thanks.

review: Needs Fixing

« Back to merge proposal