-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
[beta] Backport Clippy lint renaming / regrouping #90561
Conversation
…lip1995 Move if_then_panic to pedantic and rename to manual_assert
[beta] Backport: Move if_then_panic to pedantic and rename to manual_assert Backport of rust-lang#7810 cc `@camsteffen` r? `@ghost` changelog: none (same release)
Some changes occurred in src/tools/clippy. cc @rust-lang/clippy |
(rust-highfive has picked a reviewer for you, use r? to override) |
|
@bors r+ rollup=never This seems OK. I'd probably prefer to delete the newly introduced lint, reverting the patch introducing it, (rather than backporting the rename), but the changes are small enough that it doesn't seem that bad -- and the deletion would probably be closer to a revert but still likely to be larger, so not necessarily better. |
📌 Commit 4cd7ad0 has been approved by |
⌛ Testing commit 4cd7ad0 with merge e3d230e22989fe1400cd6d5d2c0397191912ab84... |
A job failed! Check out the build log: (web) (plain) Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
@bors retry |
⌛ Testing commit 4cd7ad0 with merge b3ccf6469708d3839b2649435bb16e9eb4dd7575... |
💔 Test failed - checks-actions |
A job failed! Check out the build log: (web) (plain) Click to see the possible cause of the failure (guessed by this bot)
|
⌛ Testing commit 4cd7ad0 with merge 4ff025b286415f50b401a161eb64f1d6a48d611f... |
💔 Test failed - checks-actions |
A job failed! Check out the build log: (web) (plain) Click to see the possible cause of the failure (guessed by this bot)
|
This seems like a general certificate issue with the beta branch. I'm not sure what to do here. |
We need to backport one of the fix PRs, most likely. I'll add it to my list. |
@bors r- |
let's try this again now that the beta backports with the python cert stuff was merged |
📌 Commit 4cd7ad0 has been approved by |
☀️ Test successful - checks-actions |
This backports PR rust-lang/rust-clippy#7810 to beta, where a lint added in the 1.57 (current beta) release cycle was renamed shortly after the beta cutoff. We want to backport this, so we don't have unnecessary
lint_store.registered_renamed
calls in the Clippy codebase forever.(The formatting changes were necessary to pass this backport in the Clippy CI, removing it in a subtree sync would be a bit painful...)
cc @camsteffen