Code review comment for lp:~dweimer/swift/lp837428

Revision history for this message
gholt (gholt) wrote :

I've proposed a merge to your branch that adds a test that would fail if your code wasn't present and does a quick PEP8 update. The merge proposal is at https://code.launchpad.net/~gholt/swift/lp837428/+merge/73439 and you can merge it into your branch by 'cd <branch>; bzr merge lp:~gholt/swift/lp837428' and then push when you're ready.

Also, if you could ensure you or your company has signed the CLA http://wiki.openstack.org/HowToContribute and add your name and info to http://wiki.openstack.org/Contributors

Thanks!

review: Needs Fixing

« Back to merge proposal