Code review comment for lp:~james-w/launchpad/code-imports-for-source-packages

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

The only problem I have with this is that

Header
++++++

lines should be preceded by two blank lines. There are two in the diff ('Approving an import' and 'Package Branches') that are not (one of these not being your fault, but please fix anyway).

Otherwise, great.

review: Approve

« Back to merge proposal