Skip to content

Commit

Permalink
Add PR link to pr_finish
Browse files Browse the repository at this point in the history
  • Loading branch information
olivroy committed May 18, 2024
1 parent da18e11 commit f6c0e14
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions R/pr.R
Original file line number Diff line number Diff line change
Expand Up @@ -967,22 +967,22 @@ pr_branch_delete <- function(pr) {

if (is.null(pr_ref)) {
ui_bullets(c(
"i" = "PR {.val {pr$pr_string}} originated from branch {.val {pr_remref}},
"i" = "PR {.href [{.val {pr$pr_string}}]({pr$pr_html_url})} originated from branch {.val {pr_remref}},

Check warning on line 970 in R/pr.R

View check run for this annotation

Codecov / codecov/patch

R/pr.R#L970

Added line #L970 was not covered by tests
which no longer exists."
))
return(invisible(FALSE))
}

if (is.na(pr$pr_merged_at)) {
ui_bullets(c(
"i" = "PR {.val {pr$pr_string}} is unmerged, we will not delete the
"i" = "PR {.href [{.val {pr$pr_string}}]({pr$pr_html_url})} is unmerged, we will not delete the

Check warning on line 978 in R/pr.R

View check run for this annotation

Codecov / codecov/patch

R/pr.R#L978

Added line #L978 was not covered by tests
remote branch {.val {pr_remref}}."
))
return(invisible(FALSE))
}

ui_bullets(c(
"v" = "PR {.val {pr$pr_string}} has been merged, deleting remote branch
"v" = "PR {.href [{.val {pr$pr_string}}]({pr$pr_html_url})} has been merged, deleting remote branch

Check warning on line 985 in R/pr.R

View check run for this annotation

Codecov / codecov/patch

R/pr.R#L985

Added line #L985 was not covered by tests
{.val {pr_remref}}."
))
# TODO: tryCatch here?
Expand Down

0 comments on commit f6c0e14

Please sign in to comment.