Skip to content

Commit

Permalink
Auto merge of #17761 - RalfJung:josh-roudntrip-error, r=lnicola
Browse files Browse the repository at this point in the history
when josh-proxy screws up the roundtrip, say what the involved commits are

Helps debugging rust-lang/rust-analyzer#17733
  • Loading branch information
bors committed Aug 1, 2024
2 parents bf5844e + 70805f9 commit 9ec4844
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/tools/rust-analyzer/xtask/src/release.rs
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,10 @@ impl flags::RustcPush {
let head = cmd!(sh, "git rev-parse HEAD").read()?;
let fetch_head = cmd!(sh, "git rev-parse FETCH_HEAD").read()?;
if head != fetch_head {
bail!("Josh created a non-roundtrip push! Do NOT merge this into rustc!");
bail!(
"Josh created a non-roundtrip push! Do NOT merge this into rustc!\n\
Expected {head}, got {fetch_head}."
);
}
println!("Confirmed that the push round-trips back to rust-analyzer properly. Please create a rustc PR:");
// https://github.com/github-linguist/linguist/compare/master...octocat:linguist:master
Expand Down

0 comments on commit 9ec4844

Please sign in to comment.