Code review comment for lp://staging/~soren/nova/lp733609

Revision history for this message
Soren Hansen (soren) wrote :

> Patch looks really good.
>
> Received a few pep8 nits from the test-suite:
> http://paste.openstack.org/show/921/

*sigh* Fixed. I keep forgetting to run pep8 manually (because I have that test case that fails due to non-ascii stuff in the help text, so run_tests.sh skips pep8). I've given myself a good shouting at. Apologies. :(

« Back to merge proposal