Skip to content

Commit 9efb478

Browse files
authored
Merge pull request #1308 from rylev/always-next-steps
Always include next steps in GitHub message
2 parents d65ed4b + 6c067b2 commit 9efb478

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

site/src/github.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -612,7 +612,7 @@ async fn summarize_run(ctxt: &SiteCtxt, commit: QueuedCommit, is_master_commit:
612612
if !primary.is_relevant() && !secondary.is_relevant() {
613613
write!(
614614
&mut message,
615-
"This benchmark run did not return any relevant results.\n\n{footer}"
615+
"This benchmark run did not return any relevant results.\n"
616616
)
617617
.unwrap();
618618
} else {
@@ -630,12 +630,12 @@ async fn summarize_run(ctxt: &SiteCtxt, commit: QueuedCommit, is_master_commit:
630630
.unwrap();
631631

632632
write_summary_table(&primary, &secondary, true, &mut message);
633-
634-
let direction = primary.direction().or(secondary.direction());
635-
let next_steps = next_steps(primary, secondary, direction, is_master_commit);
636-
write!(&mut message, "\n{footer}\n{next_steps}").unwrap();
637633
}
638634

635+
let direction = primary.direction().or(secondary.direction());
636+
let next_steps = next_steps(primary, secondary, direction, is_master_commit);
637+
write!(&mut message, "\n{footer}\n{next_steps}").unwrap();
638+
639639
message
640640
}
641641

0 commit comments

Comments
 (0)