Reviewers: mp+203942_code.launchpad.net,
Message: Please take a look.
Description: Minor fixes to code comments and documentation
Also fixed a possible encoding problem in error handling.
The get_admin_secret function and its test case are just moved, no need to re-review.
Tests: `make check`. No QA.
https://code.launchpad.net/~frankban/juju-quickstart/minor-fixes-0/+merge/203942
(do not edit description out of merge proposal)
Please review this at https://codereview.appspot.com/58680043/
Affected files (+58, -48 lines): M HACKING.rst A [revision details] M quickstart/app.py M quickstart/manage.py M quickstart/tests/test_app.py M quickstart/tests/test_manage.py
« Back to merge proposal
Reviewers: mp+203942_ code.launchpad. net,
Message:
Please take a look.
Description:
Minor fixes to code comments and documentation
Also fixed a possible encoding problem in error handling.
The get_admin_secret function and its test case are just moved, no
need to re-review.
Tests: `make check`.
No QA.
https:/ /code.launchpad .net/~frankban/ juju-quickstart /minor- fixes-0/ +merge/ 203942
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/58680043/
Affected files (+58, -48 lines): manage. py tests/test_ app.py tests/test_ manage. py
M HACKING.rst
A [revision details]
M quickstart/app.py
M quickstart/
M quickstart/
M quickstart/