Merge lp:~james-w/tarmac/break-up-do-merges into lp:tarmac
Status: | Superseded |
---|---|
Proposed branch: | lp:~james-w/tarmac/break-up-do-merges |
Merge into: | lp:tarmac |
Prerequisite: | lp:~james-w/tarmac/commit-message |
Diff against target: |
454 lines (+206/-192) 1 file modified
tarmac/bin/commands.py (+206/-192) |
To merge this branch: | bzr merge lp:~james-w/tarmac/break-up-do-merges |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paul Hummer | Pending | ||
Review via email: mp+144809@code.launchpad.net |
This proposal has been superseded by a proposal from 2013-01-28.
Description of the change
This branch breaks up the _do_merges method a little, as it was huge.
I tried to keep changes to a minimum while moving the code around, but it
used "continue" for flow control, so I had to change it a little to
use return values.
(from the u1 fork)
Unmerged revisions
- 425. By James Westby
-
Merged commit-message into break-up-do-merges.
- 424. By James Westby
-
Merged commit-message into break-up-do-merges.
- 423. By James Westby
-
Merge commit-message.
- 422. By James Westby
-
Merged commit-message into break-up-do-merges.
- 421. By James Westby
-
Merge commit-message.
- 420. By James Westby
-
Merge commit message.
- 419. By James Westby
-
Properly check the statuses of the merges when merging just one branch.
- 418. By James Westby
-
Remove the option that isn't defined yet.
- 417. By James Westby
-
Break up the very large do_merges method somewhat.
- 416. By James Westby
-
Add the commit message plugin that sets [r=<reviewer>] and bug info.