Skip to content

Use a different lint for the fix_only_once_for_duplicates test #15713

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
Jun 28, 2025

Conversation

Urgau
Copy link
Member

@Urgau Urgau commented Jun 28, 2025

What does this PR try to resolve?

This PR changes the triggering rustc lint in the fix_only_once_for_duplicates test from unused_must_use to forgetting_references, because we are changing in rust-lang/rust#143030 the interaction between that lint and macros.

This is required as it blocking the rust PR, rust-lang/rust#143030 (comment).

How to test and review this PR?

Look at the test changes.

@rustbot
Copy link
Collaborator

rustbot commented Jun 28, 2025

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 Jun 28, 2025
@epage epage added this pull request to the merge queue Jun 28, 2025
Merged via the queue into rust-lang:master with commit 930b4f6 Jun 28, 2025
24 checks passed
bors added a commit to rust-lang/rust that referenced this pull request Jun 29, 2025
Update cargo

7 commits in 409fed7dc1553d49cb9a8c0637d12d65571346ce..930b4f62cfcd1f0eabdb30a56d91bf6844b739bf
2025-06-23 15:55:04 +0000 to 2025-06-28 14:58:43 +0000
- Use a different lint for the `fix_only_once_for_duplicates` test (rust-lang/cargo#15713)
- chore: bump to 0.91.0; update changelog (rust-lang/cargo#15710)
- Add `http.proxy-cainfo` config for proxy certs (rust-lang/cargo#15374)
- chore(deps): update msrv (3 versions) to v1.86 (rust-lang/cargo#15709)
- chore(deps): update msrv (1 version) to v1.88 (rust-lang/cargo#15706)
- Rework `cargo-test-support` & `testsuite` to use `CARGO_BIN_EXE_*` for Cargo (rust-lang/cargo#15692)
- fix: Expand error messages around path dependency on `cargo package` and `cargo publish` (rust-lang/cargo#15705)
@rustbot rustbot added this to the 1.90.0 milestone Jun 29, 2025
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Jun 29, 2025
Update cargo

7 commits in 409fed7dc1553d49cb9a8c0637d12d65571346ce..930b4f62cfcd1f0eabdb30a56d91bf6844b739bf
2025-06-23 15:55:04 +0000 to 2025-06-28 14:58:43 +0000
- Use a different lint for the `fix_only_once_for_duplicates` test (rust-lang/cargo#15713)
- chore: bump to 0.91.0; update changelog (rust-lang/cargo#15710)
- Add `http.proxy-cainfo` config for proxy certs (rust-lang/cargo#15374)
- chore(deps): update msrv (3 versions) to v1.86 (rust-lang/cargo#15709)
- chore(deps): update msrv (1 version) to v1.88 (rust-lang/cargo#15706)
- Rework `cargo-test-support` & `testsuite` to use `CARGO_BIN_EXE_*` for Cargo (rust-lang/cargo#15692)
- fix: Expand error messages around path dependency on `cargo package` and `cargo publish` (rust-lang/cargo#15705)
epage added a commit to epage/cargo that referenced this pull request Jun 30, 2025
github-merge-queue bot pushed a commit that referenced this pull request Jun 30, 2025
### What does this PR try to resolve?

This unblocks Red CI, see [#t-cargo > Red CI: rustfix @
💬](https://rust-lang.zulipchat.com/#narrow/channel/246057-t-cargo/topic/Red.20CI.3A.20rustfix/near/526449337)

### How to test and review this PR?

Inspired by #15713
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.

3 participants