Code review comment for lp:~frankban/juju-quickstart/prepare-new-release

Revision history for this message
Francesco Banconi (frankban) wrote :

Reviewers: mp+207145_code.launchpad.net,

Message:
Please take a look.

Description:
Minor changes in preparation for release.

Bump version up to 1.1.0.
Re-organize the "requires sudo" code.
Restore 100% unit test coverage (compulsive mode on).
Update license headers of modified files.

Tests: `make check`
QA: use quickstart as usual, bootstrapping
both ec2 and local environments.

https://code.launchpad.net/~frankban/juju-quickstart/prepare-new-release/+merge/207145

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/65970043/

Affected files (+93, -71 lines):
   A [revision details]
   M quickstart/__init__.py
   M quickstart/app.py
   M quickstart/manage.py
   M quickstart/settings.py
   M quickstart/tests/test_app.py
   M quickstart/tests/test_manage.py
   M quickstart/tests/test_utils.py
   M quickstart/utils.py

« Back to merge proposal