Skip to content

Commit f923ffd

Browse files
committed
Remove unneeded wrap in reapproval comment message
1 parent 255dbe6 commit f923ffd

File tree

1 file changed

+9
-15
lines changed

1 file changed

+9
-15
lines changed

src/bors/handlers/pr_events.rs

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -312,8 +312,7 @@ async fn notify_of_pushed_pr(
312312
cancel_message: Option<String>,
313313
) -> anyhow::Result<()> {
314314
let mut comment = format!(
315-
r#":warning: A new commit `{head_sha}` was pushed to the branch, the
316-
PR will need to be re-approved."#
315+
r#":warning: A new commit `{head_sha}` was pushed to the branch, the PR will need to be re-approved."#
317316
);
318317

319318
if let Some(message) = cancel_message {
@@ -401,10 +400,7 @@ mod tests {
401400

402401
insta::assert_snapshot!(
403402
tester.get_next_comment_text(()).await?,
404-
@r"
405-
:warning: A new commit `pr-1-commit-1` was pushed to the branch, the
406-
PR will need to be re-approved.
407-
"
403+
@":warning: A new commit `pr-1-commit-1` was pushed to the branch, the PR will need to be re-approved."
408404
);
409405
tester.get_pr_copy(()).await.expect_unapproved();
410406
Ok(())
@@ -714,13 +710,12 @@ mod tests {
714710
.await?;
715711
tester.push_to_pr(()).await?;
716712
insta::assert_snapshot!(tester.get_next_comment_text(()).await?, @r"
717-
:warning: A new commit `pr-1-commit-1` was pushed to the branch, the
718-
PR will need to be re-approved.
713+
:warning: A new commit `pr-1-commit-1` was pushed to the branch, the PR will need to be re-approved.
719714
720-
Auto build cancelled due to push. Cancelled workflows:
715+
Auto build cancelled due to push. Cancelled workflows:
721716
722-
- https://github.com/rust-lang/borstest/actions/runs/123
723-
");
717+
- https://github.com/rust-lang/borstest/actions/runs/123
718+
");
724719
Ok(())
725720
})
726721
.await;
@@ -742,11 +737,10 @@ mod tests {
742737
tester.workflow_start(tester.auto_branch().await).await?;
743738
tester.push_to_pr(()).await?;
744739
insta::assert_snapshot!(tester.get_next_comment_text(()).await?, @r"
745-
:warning: A new commit `pr-1-commit-1` was pushed to the branch, the
746-
PR will need to be re-approved.
740+
:warning: A new commit `pr-1-commit-1` was pushed to the branch, the PR will need to be re-approved.
747741
748-
Auto build cancelled due to push. It was not possible to cancel some workflows.
749-
");
742+
Auto build cancelled due to push. It was not possible to cancel some workflows.
743+
");
750744
Ok(())
751745
})
752746
.await;

0 commit comments

Comments
 (0)