Skip to content

[beta] Clippy backports #133299

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 2 commits into from
Nov 25, 2024
Merged

[beta] Clippy backports #133299

merged 2 commits into from
Nov 25, 2024

Conversation

flip1995
Copy link
Member

r? @Mark-Simulacrum

Backports:

The first is just a regrouping to a allow-by-default group, as we figured that the lint would be too noisy as a warn-by-default lint. That lint was added last release cycle, so we want to do the re-grouping before it hits stable.

The second is a bug fix for &raw references that are already stable in 1.82, but we don't want to wait another release cycle for the fix to land on stable.

Both commits are already synced to the current master branch.

bors added 2 commits November 21, 2024 16:30
…ge-category, r=llogiq

Change the category of `manual_is_power_of_two` to `pedantic`

Fixes rust-lang#13547.

The value being checked might be a bit flag, suggesting `is_power_of_two` for it would make the code unreadable.

changelog: [`manual_is_power_of_two`]: Change the category to `pedantic`
…rednet

borrow_deref_ref: do not trigger on `&raw` references

changelog: [`borrow_deref_ref`]: do not trigger on `&raw` references

Fix rust-lang#13584
@rustbot
Copy link
Collaborator

rustbot commented Nov 21, 2024

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against beta. Please double check that you specified the right target!

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 21, 2024
@rustbot
Copy link
Collaborator

rustbot commented Nov 21, 2024

Some changes occurred in src/tools/clippy

cc @rust-lang/clippy

@Mark-Simulacrum
Copy link
Member

@bors r+ rollup=never p=10

@bors
Copy link
Collaborator

bors commented Nov 24, 2024

📌 Commit 5f4dd1d has been approved by Mark-Simulacrum

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 Nov 24, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 24, 2024
…k-Simulacrum

[beta] Clippy backports

r? `@Mark-Simulacrum`

Backports:
- rust-lang/rust-clippy#13553
- rust-lang/rust-clippy#13600

The first is just a regrouping to a allow-by-default group, as we figured that the lint would be too noisy as a warn-by-default lint. That lint was added last release cycle, so we want to do the re-grouping before it hits stable.

The second is a bug fix for `&raw` references that are already stable in `1.82`, but we don't want to wait another release cycle for the fix to land on stable.

Both commits are already synced to the current `master branch`.
@bors
Copy link
Collaborator

bors commented Nov 24, 2024

⌛ Testing commit 5f4dd1d with merge 420638e...

@rust-log-analyzer
Copy link
Collaborator

The job dist-x86_64-illumos failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@bors
Copy link
Collaborator

bors commented Nov 24, 2024

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Nov 24, 2024
@Mark-Simulacrum
Copy link
Member

@bors retry

Timed out?

@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 Nov 25, 2024
@bors
Copy link
Collaborator

bors commented Nov 25, 2024

⌛ Testing commit 5f4dd1d with merge f9c2695...

@bors
Copy link
Collaborator

bors commented Nov 25, 2024

☀️ Test successful - checks-actions
Approved by: Mark-Simulacrum
Pushing f9c2695 to beta...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Nov 25, 2024
@bors bors merged commit f9c2695 into rust-lang:beta Nov 25, 2024
7 checks passed
@rustbot rustbot added this to the 1.83.0 milestone Nov 25, 2024
@flip1995 flip1995 deleted the clippy-beta-backport branch November 25, 2024 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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