Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(321)

Issue 37150043: Envs management: code reorganization.

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 5 months ago by frankban
Modified:
10 years, 5 months ago
Reviewers:
mp+197709, gary.poster
Visibility:
Public.

Description

Envs management: code reorganization. This branch includes code reorg as a preparation for the upcoming envs management work. It's just about mechanically moving code to the new models package, except for a minor fix to test_init_environment in test_manage.py. https://code.launchpad.net/~frankban/juju-quickstart/env-management-1/+merge/197709 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Envs management: code reorganization. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+269 lines, -202 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M quickstart/manage.py View 3 chunks +6 lines, -3 lines 0 comments Download
A quickstart/models/__init__.py View 1 chunk +17 lines, -0 lines 0 comments Download
M quickstart/models/charms.py View 0 chunks +-1 lines, --1 lines 0 comments Download
A quickstart/models/envs.py View 1 chunk +104 lines, -0 lines 0 comments Download
A quickstart/tests/models/__init__.py View 0 chunks +-1 lines, --1 lines 0 comments Download
M quickstart/tests/models/test_charms.py View 1 chunk +1 line, -1 line 0 comments Download
A quickstart/tests/models/test_envs.py View 1 chunk +136 lines, -0 lines 0 comments Download
M quickstart/tests/test_manage.py View 2 chunks +3 lines, -4 lines 0 comments Download
M quickstart/tests/test_utils.py View 2 chunks +0 lines, -111 lines 0 comments Download
M quickstart/utils.py View 3 chunks +2 lines, -85 lines 0 comments Download

Messages

Total messages: 3
frankban
Please take a look.
10 years, 5 months ago (2013-12-04 13:20:09 UTC) #1
gary.poster
LGTM. Thank you!
10 years, 5 months ago (2013-12-04 13:26:50 UTC) #2
frankban
10 years, 5 months ago (2013-12-04 13:29:45 UTC) #3
*** Submitted:

Envs management: code reorganization.

This branch includes code reorg as
a preparation for the upcoming envs
management work.

It's just about mechanically moving code 
to the new models package, except for a 
minor fix to test_init_environment in 
test_manage.py.

R=gary.poster
CC=
https://codereview.appspot.com/37150043
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b