Code review comment for lp:~jamalta/launchpad/destination-ppa-498643

Revision history for this message
Brad Crittenden (bac) wrote :

Hi Jamal,

Your fix looks good. In your diff at line 9, please change the line back to have a 4 space indentation. You can then bring the following line up as it will just be 77 characters.

Also you agreed to fix the import violation exposed by running your test. Thanks.

review: Approve (code)

« Back to merge proposal