Code review comment for lp:~jimbaker/pyjuju/unit-rel-lifecycle-start-invoker

Revision history for this message
Jim Baker (jimbaker) wrote :

Thanks, changed the minors and pushed accordingly

https://codereview.appspot.com/6131061/diff/3001/juju/unit/tests/test_lifecycle.py
File juju/unit/tests/test_lifecycle.py (right):

https://codereview.appspot.com/6131061/diff/3001/juju/unit/tests/test_lifecycle.py#newcode901
juju/unit/tests/test_lifecycle.py:901: for i in xrange(1, 3):
Pure habit here, fixed by using range instead

https://codereview.appspot.com/6131061/diff/3001/juju/unit/tests/test_lifecycle.py#newcode1331
juju/unit/tests/test_lifecycle.py:1331: for i in xrange(1, 5):
Ditto fix

https://codereview.appspot.com/6131061/

« Back to merge proposal