Merge lp://staging/~doanac/ubuntu-ci-services-itself/ppa-migrations into lp://staging/ubuntu-ci-services-itself

Proposed by Andy Doan
Status: Merged
Approved by: Vincent Ladeuil
Approved revision: 341
Merged at revision: 342
Proposed branch: lp://staging/~doanac/ubuntu-ci-services-itself/ppa-migrations
Merge into: lp://staging/ubuntu-ci-services-itself
Diff against target: 60 lines (+40/-0)
2 files modified
ppa-assigner/ppa_assigner/migrations/0001_initial.py (+38/-0)
ppa-assigner/ppa_assigner/tests.py (+2/-0)
To merge this branch: bzr merge lp://staging/~doanac/ubuntu-ci-services-itself/ppa-migrations
Reviewer Review Type Date Requested Status
Vincent Ladeuil (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+210034@code.staging.launchpad.net

Commit message

ppa-assigner: add initial south migration

Description of the change

we need to get a django-south migration in place for the ppa-assigner for phase-0

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:341
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/337/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/337/rebuild

review: Approve (continuous-integration)
Revision history for this message
Vincent Ladeuil (vila) wrote :

Seems straight forward. Can we add some documentation on how this migration files are generated ? That may be common knowledge and well documented elsewhere so even some pointers to the relevant doc would be enough.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches