rustup https://github.com/rust-lang/rust/pull/72342, allow unused_crate_dependencies

[matthiaskrgr]

changelog: none

[rust-highfive]

r? @phansch

(rust_highfive has picked a reviewer for you, use r? to override)

[flip1995]

@bors r+

[bors]

📌 Commit 3089c3b has been approved by flip1995

[bors]

⌛ Testing commit 3089c3b with merge 2a2208f...

[phansch]

@matthiaskrgr Huh, why would the lint have to be allowed only in these two specific files?

[matthiaskrgr]

I have no idea to be honest :/

[phansch]

Weird.. I will try and see if I can reproduce this in the rustc testsuite 🤷‍♂️

[matthiaskrgr]

I wondered if it is perhaps related to the unused lint group but one test allows it while the other one warns on it so I dunno ...

[bors]

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 2a2208f to master...

[phansch]

I wondered if it is perhaps related to the unused lint group but one test allows it while the other one warns on it so I dunno ...

Ahh, right 🤦 tests/ui/cognitive_complexity_attr_used.rs was the only of our tests with #![warn(unused)] that's why it triggered only there. I believe the #![allow(unused_crate_dependencies)] in tests/ui/cognitive_complexity.rs is not required either?

[mati865]

Soon will be fixed by rust-lang/rust#72702