fix: show branch name in push events in HTML reports
We used to parse the ref name to find out the branch name, when creating a trigger message. The CI events now do that for us. This meant that parsing the branch name failed, and resulted in the empty string.
Fix this by using already-parsed branch name from the CI event instead of trying to parse it again.
Signed-off-by: Lars Wirzenius liw@liw.fi
2 files changed
+6
-47
70f1ba7d
→
67c215c9
modified src/broker.rs
@@ -86,12 +86,12 @@ impl Broker {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
modified src/msg.rs
@@ -161,7 +161,7 @@ impl<'a> RequestBuilder<'a> {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -217,7 +217,7 @@ impl<'a> RequestBuilder<'a> {
|
|
|
|
|
|
| - | |
| + | |
|
|
|
|
|
@@ -945,9 +945,7 @@ pub mod trigger_from_ci_event_tests {
|
|
|
|
|
|
| - | |
| - | |
| - | |
| + | |
|
|
|
|
|
@@ -996,9 +994,7 @@ pub mod trigger_from_ci_event_tests {
|
|
|
|
|
|
| - | |
| - | |
| - | |
| + | |
|
|
|
|
|
@@ -1176,40 +1172,3 @@ pub mod trigger_from_ci_event_tests {
|
|
|
|
|
|
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |
| - | |