pyjuju

Code review comment for lp:~fwereade/pyjuju/fix-charm-upgrade

  1. fix-charm-upgrade
  2. Merge into trunk
Revision history for this message
Jim Baker (jimbaker) wrote :

+1, looks good to me. The only thing:

$ pep8 juju/unit
juju/unit/tests/test_lifecycle.py:830:1: E303 too many blank lines (3)
juju/unit/tests/test_workflow.py:619:80: E501 line too long (80 characters)

review: Approve

« Back to merge proposal