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

Do not expand macros in equatable_if_let suggestion #7788

Merged
merged 1 commit into from
Oct 13, 2021

Conversation

flip1995
Copy link
Member

@flip1995 flip1995 commented Oct 8, 2021

Fixes #7781

Let's use Hacktoberfest as a motivation to start contributing PRs myself again :)

changelog: [equatable_if_let]: No longer expands macros in the suggestion

@rust-highfive
Copy link

r? @Manishearth

(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 Oct 8, 2021
@giraffate
Copy link
Contributor

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Oct 13, 2021

📌 Commit 28fb591 has been approved by giraffate

@bors
Copy link
Contributor

bors commented Oct 13, 2021

⌛ Testing commit 28fb591 with merge 9e835e0...

bors added a commit that referenced this pull request Oct 13, 2021
Do not expand macros in equatable_if_let suggestion

Fixes #7781

Let's use Hacktoberfest as a motivation to start contributing PRs myself again :)

changelog: [`equatable_if_let`]: No longer expands macros in the suggestion
@bors
Copy link
Contributor

bors commented Oct 13, 2021

💔 Test failed - checks-action_test

@flip1995
Copy link
Member Author

@bors retry (macOS error looks spurious)

@bors
Copy link
Contributor

bors commented Oct 13, 2021

⌛ Testing commit 28fb591 with merge 072ff0d...

bors added a commit that referenced this pull request Oct 13, 2021
Do not expand macros in equatable_if_let suggestion

Fixes #7781

Let's use Hacktoberfest as a motivation to start contributing PRs myself again :)

changelog: [`equatable_if_let`]: No longer expands macros in the suggestion
@bors
Copy link
Contributor

bors commented Oct 13, 2021

💔 Test failed - checks-action_test

@flip1995
Copy link
Member Author

@bors retry (windows GHA incident should be fixed now)

@bors
Copy link
Contributor

bors commented Oct 13, 2021

⌛ Testing commit 28fb591 with merge 57dc034...

@bors
Copy link
Contributor

bors commented Oct 13, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: giraffate
Pushing 57dc034 to master...

@bors bors merged commit 57dc034 into rust-lang:master Oct 13, 2021
@flip1995 flip1995 deleted the eq_if_let_sugg branch October 13, 2021 12:18
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.

equatable_if_let should not expand macros in suggestion
5 participants