Code review comment for lp:~jtv/launchpad/bug-611217

Revision history for this message
Māris Fogels (mars) wrote :

Hi Jeroen,

This change looks great, r=mars.

I think you have one stray if-statement around line 72 of the diff: you can probably collapse that into a straight if/elif instead of nested statements.

Looks good!

review: Approve

« Back to merge proposal