Merge lp://staging/~frankban/juju-quickstart/jujuutils into lp://staging/juju-quickstart
Proposed by
Francesco Banconi
Status: | Merged |
---|---|
Merged at revision: | 120 |
Proposed branch: | lp://staging/~frankban/juju-quickstart/jujuutils |
Merge into: | lp://staging/juju-quickstart |
Diff against target: |
1480 lines (+713/-599) 8 files modified
quickstart/app.py (+6/-4) quickstart/jujutools.py (+127/-0) quickstart/manage.py (+3/-2) quickstart/models/bundles.py (+118/-0) quickstart/tests/models/test_bundles.py (+254/-0) quickstart/tests/test_jujutools.py (+204/-0) quickstart/tests/test_utils.py (+0/-397) quickstart/utils.py (+1/-196) |
To merge this branch: | bzr merge lp://staging/~frankban/juju-quickstart/jujuutils |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email:
|
Description of the change
Refactor/split utils code.
Move code from utils to the new jujutools and models/bundles modules.
This branch only include moved code and it's done
in preparation for the Juju API new login work.
To post a comment you must log in.
Reviewers: mp+249073_ code.launchpad. net,
Message:
Please take a look.
Description:
Refactor/split utils code.
Move code from utils to the new jujutools and models/bundles modules.
This branch only include moved code and it's done
in preparation for the Juju API new login work.
https:/ /code.launchpad .net/~frankban/ juju-quickstart /jujuutils/ +merge/ 249073
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/204770043/
Affected files (+715, -599 lines): jujutools. py manage. py models/ bundles. py tests/models/ test_bundles. py tests/test_ jujutools. py tests/test_ utils.py
A [revision details]
M quickstart/app.py
A quickstart/
M quickstart/
A quickstart/
A quickstart/
A quickstart/
M quickstart/
M quickstart/utils.py