diff --git a/hive-forge/src/verbs/pr_merge.rs b/hive-forge/src/verbs/pr_merge.rs index 89a22134..83548244 100644 --- a/hive-forge/src/verbs/pr_merge.rs +++ b/hive-forge/src/verbs/pr_merge.rs @@ -82,10 +82,10 @@ pub fn run(client: &Client, args: Args) -> Result<()> { .send()?; if pull.merged.unwrap_or(false) { - bail!("pr-merge: PR #{} is already merged", args.number); + bail!("pr merge: PR #{} is already merged", args.number); } if pull.state == Some(StateType::Closed) { - bail!("pr-merge: PR #{} is closed", args.number); + bail!("pr merge: PR #{} is closed", args.number); } if !args.force { @@ -139,13 +139,13 @@ fn check_ready(client: &Client, repo: &str, number: u64, pull: &PullRequest) -> // rather than an oversight, so don't tell the caller to just // undraft it if it isn't theirs to undraft. Some(false) if pull.draft.unwrap_or(false) => bail!( - "pr-merge: PR #{number} is a draft, not mergeable. Pass --force, or check with whoever set draft before un-drafting it." + "pr merge: PR #{number} is a draft, not mergeable. Pass --force, or check with whoever set draft before un-drafting it." ), Some(false) => bail!( - "pr-merge: PR #{number} is not mergeable. Check `pr status --pr {number}` for detail, or pass --force." + "pr merge: PR #{number} is not mergeable. Check `pr status {number}` for detail, or pass --force." ), None => bail!( - "pr-merge: PR #{number} mergeability is still being computed. Retry shortly, or pass --force." + "pr merge: PR #{number} mergeability is still being computed. Retry shortly, or pass --force." ), } @@ -156,7 +156,7 @@ fn check_ready(client: &Client, repo: &str, number: u64, pull: &PullRequest) -> if !combined.statuses.is_empty() && combined.state != "success" { let state = &combined.state; bail!( - "pr-merge: PR #{number} CI is not green (state: {state}). Wait for green, or pass --force." + "pr merge: PR #{number} CI is not green (state: {state}). Wait for green, or pass --force." ); } } @@ -173,7 +173,7 @@ fn check_ready(client: &Client, repo: &str, number: u64, pull: &PullRequest) -> .collect(); if !blockers.is_empty() { bail!( - "pr-merge: PR #{number} has changes requested by {}. Resolve the review, or pass --force.", + "pr merge: PR #{number} has changes requested by {}. Resolve the review, or pass --force.", blockers.join(", ") ); }