Skip to content
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

Fix explicit_auto_deref fp #9813

Merged
merged 1 commit into from
Nov 9, 2022
Merged

Fix explicit_auto_deref fp #9813

merged 1 commit into from
Nov 9, 2022

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Nov 7, 2022

fixes #9763
fixes #9811

changelog: explicit_auto_deref: Don't lint when the target type is a projection with generic arguments

@rust-highfive
Copy link

r? @xFrednet

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 7, 2022
@xFrednet
Copy link
Member

xFrednet commented Nov 9, 2022

LGTM, thank you for the fix!

@bors r+

@bors
Copy link
Collaborator

bors commented Nov 9, 2022

📌 Commit 5b1e445 has been approved by xFrednet

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Nov 9, 2022

⌛ Testing commit 5b1e445 with merge 432baf7...

@bors
Copy link
Collaborator

bors commented Nov 9, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: xFrednet
Pushing 432baf7 to master...

@bors bors merged commit 432baf7 into rust-lang:master Nov 9, 2022
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.

explicit_auto_deref lint suggests broken code explicit_auto_deref false positive
4 participants