Merge lp://qastaging/~dobey/tarmac/merge-web-link into lp://qastaging/tarmac
Proposed by
dobey
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | dobey | ||||
Approved revision: | 428 | ||||
Merged at revision: | 428 | ||||
Proposed branch: | lp://qastaging/~dobey/tarmac/merge-web-link | ||||
Merge into: | lp://qastaging/tarmac | ||||
Diff against target: |
158 lines (+19/-46) 5 files modified
tarmac/bin/commands.py (+2/-4) tarmac/plugins/commitmessage.py (+1/-2) tarmac/plugins/tests/test_commitmessage.py (+4/-2) tarmac/tests/test_commands.py (+12/-28) tarmac/utility.py (+0/-10) |
||||
To merge this branch: | bzr merge lp://qastaging/~dobey/tarmac/merge-web-link | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mike McCracken | Approve | ||
Review via email: mp+207347@code.qastaging.launchpad.net |
Commit message
Use proposal.web_link directly instead of a regex.
To post a comment you must log in.
looks good, tests pass for me