Skip to content

internal: switch remaining OpQueues to use named structs#18216

Merged
bors merged 1 commit intorust-lang:masterfrom
davidbarsky:davidbarsky/push-ustyovtqrpoq
Oct 14, 2024
Merged

internal: switch remaining OpQueues to use named structs#18216
bors merged 1 commit intorust-lang:masterfrom
davidbarsky:davidbarsky/push-ustyovtqrpoq

Conversation

@davidbarsky
Copy link
Contributor

Building atop of #18195, I switched GlobalState::fetch_build_data_queue to use a struct instead of a tuple.

(I didn't switch fetch_proc_macros_queue to not return a bool, as the return value is only used in one spot.)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 30, 2024
@Veykril
Copy link
Member

Veykril commented Oct 14, 2024

bors merge

@Veykril
Copy link
Member

Veykril commented Oct 14, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Oct 14, 2024

📌 Commit bc8b860 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 14, 2024

⌛ Testing commit bc8b860 with merge 22928e7...

bors added a commit that referenced this pull request Oct 14, 2024
…eykril

internal: switch remaining OpQueues to use named structs

Building atop of #18195, I switched `GlobalState::fetch_build_data_queue` to use a struct instead of a tuple.

(I didn't switch `fetch_proc_macros_queue` to not return a bool, as the return value is only used in one spot.)
@bors
Copy link
Contributor

bors commented Oct 14, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 22928e7 to master...

@bors
Copy link
Contributor

bors commented Oct 14, 2024

👀 Test was successful, but fast-forwarding failed: 422 Update is not a fast forward

@davidbarsky davidbarsky force-pushed the davidbarsky/push-ustyovtqrpoq branch from bc8b860 to 501ef0e Compare October 14, 2024 14:04
@davidbarsky
Copy link
Contributor Author

@bors r+

@bors
Copy link
Contributor

bors commented Oct 14, 2024

📌 Commit 501ef0e has been approved by davidbarsky

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 14, 2024

⌛ Testing commit 501ef0e with merge afe015b...

@bors
Copy link
Contributor

bors commented Oct 14, 2024

☀️ Test successful - checks-actions
Approved by: davidbarsky
Pushing afe015b to master...

1 similar comment
@bors
Copy link
Contributor

bors commented Oct 14, 2024

☀️ Test successful - checks-actions
Approved by: davidbarsky
Pushing afe015b to master...

@bors bors merged commit afe015b into rust-lang:master Oct 14, 2024
@bors
Copy link
Contributor

bors commented Oct 14, 2024

👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.

@davidbarsky davidbarsky deleted the davidbarsky/push-ustyovtqrpoq branch October 14, 2024 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants