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
Thanks, changed the minors and pushed accordingly
https:/ /codereview. appspot. com/6131061/ diff/3001/ juju/unit/ tests/test_ lifecycle. py tests/test_ lifecycle. py (right):
File juju/unit/
https:/ /codereview. appspot. com/6131061/ diff/3001/ juju/unit/ tests/test_ lifecycle. py#newcode901 tests/test_ lifecycle. py:901: for i in xrange(1, 3):
juju/unit/
Pure habit here, fixed by using range instead
https:/ /codereview. appspot. com/6131061/ diff/3001/ juju/unit/ tests/test_ lifecycle. py#newcode1331 tests/test_ lifecycle. py:1331: for i in xrange(1, 5):
juju/unit/
Ditto fix
https:/ /codereview. appspot. com/6131061/