Skip to content

internal: switch remaining OpQueues to use named structs #18216

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 14, 2024

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
11 checks passed
@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