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

[unnecessary_mut_passed]: don't lint in macro expansions #11269

Merged
merged 1 commit into from Aug 2, 2023

Conversation

y21
Copy link
Member

@y21 y21 commented Jul 31, 2023

Fixes #11268

changelog: [unnecessary_mut_passed]: don't lint in macro expansions

@rustbot
Copy link
Collaborator

rustbot commented Jul 31, 2023

r? @Centri3

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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jul 31, 2023
@Centri3
Copy link
Member

Centri3 commented Aug 1, 2023

Thanks! @bors r+

@bors
Copy link
Collaborator

bors commented Aug 1, 2023

📌 Commit dc1e8b0 has been approved by Centri3

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Aug 1, 2023

⌛ Testing commit dc1e8b0 with merge 588c1ab...

@bors
Copy link
Collaborator

bors commented Aug 1, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Centri3
Pushing 588c1ab to master...

This was referenced Aug 1, 2023
@Centri3
Copy link
Member

Centri3 commented Aug 1, 2023

Hmm... No idea why this wasn't closed, it was merged fine. Should I just close this?

@y21
Copy link
Member Author

y21 commented Aug 1, 2023

I've seen this happen before (#11210), it took a couple hours but eventually got closed by itself. No idea why this happens

@bors bors merged commit 588c1ab into rust-lang:master Aug 2, 2023
5 checks passed
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.

unnecessary_mut_passed (probably) shouldn't lint in macro expansions
4 participants