Better test failure behavior
Bug #411569 reported by
dobey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Tarmac |
Fix Released
|
High
|
dobey | ||
Bug Description
Currently, when tests fail, Tarmac adds a comment to the merge proposal with a subject of "Failed test command" and only the output from the test command in the comment. It should be more consistent with the general Launchpad behavior, and use "Re: [Merge] %s into %s" as the subject, and provide a little more information in the comment, as well as set the status back to 'Needs review' appropriately.
Related branches
lp:~dobey/tarmac/failed-merge-status
- Paul Hummer: Approve
- Diff: None lines
| Changed in tarmac: | |
| assignee: | nobody → Rodney Dawes (dobey) |
| importance: | Undecided → Medium |
| status: | New → In Progress |
| importance: | Medium → High |
| Changed in tarmac: | |
| status: | In Progress → Fix Committed |
| Changed in tarmac: | |
| milestone: | none → tarmac-0.3 |
| Changed in tarmac: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
