Skip to content

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Jul 28, 2024

This fixes a few warnings when building cargo's testsuite on Windows due to some imports only being used inside some cfg elided tests.

@rustbot
Copy link
Collaborator

rustbot commented Jul 28, 2024

r? @epage

rustbot has assigned @epage.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 28, 2024
@ehuss ehuss force-pushed the windows-warnings branch from d3fc12e to 9ccd7c4 Compare July 28, 2024 19:59
@weihanglo
Copy link
Member

Nice catch. Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Jul 28, 2024

📌 Commit 9ccd7c4 has been approved by weihanglo

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 Jul 28, 2024
@bors
Copy link
Contributor

bors commented Jul 28, 2024

⌛ Testing commit 9ccd7c4 with merge 59c6fad...

@bors
Copy link
Contributor

bors commented Jul 28, 2024

☀️ Test successful - checks-actions
Approved by: weihanglo
Pushing 59c6fad to master...

@bors bors merged commit 59c6fad into rust-lang:master Jul 28, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 31, 2024
Update cargo

5 commits in b5d44db1daf0469b227a6211b987162a39a54730..257b72b8adfb1f2aa9916cefca67285c21666276
2024-07-26 21:27:12 +0000 to 2024-07-30 15:08:27 +0000
- chore: Update toml (rust-lang/cargo#14324)
- dont call wrap in a no-op source_id::with* (rust-lang/cargo#14318)
- Partial revert of 3085b548ecbd7d29d40c6180609ea3c224bad857 (rust-lang/cargo#14319)
- chore(deps): update msrv (rust-lang/cargo#14303)
- Fix warnings building tests on Windows (rust-lang/cargo#14315)

r? ghost
@rustbot rustbot added this to the 1.82.0 milestone Jul 31, 2024
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants