Skip to content

Conversation

lqd
Copy link
Member

@lqd lqd commented Aug 30, 2023

bors incorrectly merged an outdated version of PR #115355 (via rollup #115370):

  • it recorded r+ as approving commit 325b585, and thus merged the original revision 7762ac7
  • but the branch at the time was at commit eefa07d, so bors missed the compiler/rustc_trait_selection/src/solve/search_graph/mod.rs cleanup in commit 0e1e964 😓

Thankfully the change that bors missed was small, and this new PR corrects the situation (as I'd rather avoid having confusing multiple merge commits of PR #115355 in the git history)

r? @compiler-errors

fixes bors snafu where it merged an outdated commit and missed this
change
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver) labels Aug 30, 2023
@rustbot
Copy link
Collaborator

rustbot commented Aug 30, 2023

Some changes occurred to the core trait solver

cc @rust-lang/initiative-trait-system-refactor

@compiler-errors
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Aug 30, 2023

📌 Commit 7272552 has been approved by compiler-errors

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 30, 2023
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 30, 2023
Fix bors missing a commit when merging rust-lang#115355

bors incorrectly merged an outdated version of PR rust-lang#115355 (via rollup rust-lang#115370):
- it [recorded r+](rust-lang#115355 (comment)) as approving commit rust-lang@325b585, and thus merged the original revision rust-lang@7762ac7
- but the branch at the time was at commit rust-lang@eefa07d, so bors missed the `compiler/rustc_trait_selection/src/solve/search_graph/mod.rs` cleanup in commit rust-lang@0e1e964 😓

Thankfully the change that bors missed was small, and this new PR corrects the situation (as I'd rather avoid having confusing multiple merge commits of PR rust-lang#115355 in the git history)

r? `@compiler-errors`
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 31, 2023
…iaskrgr

Rollup of 3 pull requests

Successful merges:

 - rust-lang#115373 (Fix bors missing a commit when merging rust-lang#115355)
 - rust-lang#115378 (`ignore-cross-compile` remaining tests that run binaries)
 - rust-lang#115393 (Make `termcolor` types public in `rustc_errors`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 0b62835 into rust-lang:master Aug 31, 2023
@rustbot rustbot added this to the 1.74.0 milestone Aug 31, 2023
@lqd lqd deleted the come-on-bors branch August 31, 2023 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. WG-trait-system-refactor The Rustc Trait System Refactor Initiative (-Znext-solver)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants