Skip to content

Fix [match_single_binding] suggestion introduced an extra semicolon #10060

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Dec 29, 2022

Conversation

alex-semenyuk
Copy link
Member

@alex-semenyuk alex-semenyuk commented Dec 10, 2022

Fix #9725


changelog: [match_single_binding]: suggestion no longer introduces unneeded semicolons
#10060

@rustbot
Copy link
Collaborator

rustbot commented Dec 10, 2022

r? @Jarcho

(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 Dec 10, 2022
@alex-semenyuk alex-semenyuk changed the title Match single binding fix Fix match_single_binding suggestion introduced an extra semicolon Dec 10, 2022
@alex-semenyuk alex-semenyuk changed the title Fix match_single_binding suggestion introduced an extra semicolon Fix [match_single_binding] suggestion introduced an extra semicolon Dec 10, 2022
@alex-semenyuk alex-semenyuk force-pushed the match_single_binding_fix branch from 268c4dd to 347e798 Compare December 10, 2022 17:21
@alex-semenyuk alex-semenyuk force-pushed the match_single_binding_fix branch from 347e798 to 3b6bbf7 Compare December 10, 2022 17:36
@alex-semenyuk
Copy link
Member Author

PING

@llogiq
Copy link
Contributor

llogiq commented Dec 29, 2022

@Jarcho seems to be busy right now. Stealing this, as it looks good to merge.

@bors r+

@bors
Copy link
Contributor

bors commented Dec 29, 2022

📌 Commit 3b6bbf7 has been approved by llogiq

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Dec 29, 2022

⌛ Testing commit 3b6bbf7 with merge 22af8fe...

@bors
Copy link
Contributor

bors commented Dec 29, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing 22af8fe to master...

@bors bors merged commit 22af8fe into rust-lang:master Dec 29, 2022
# for free to join this conversation on GitHub. Already have an account? # 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.

match_single_binding suggestion introduced an extra semicolon
5 participants