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

manual_ok_err: blockify the replacement of an else if … #14240

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

samueltardieu
Copy link
Contributor

If the part being replaced is an if expression following an else, the replacement expression must be blockified.

Fix #14239

changelog: [manual_ok_err]: fix replacement expression if it follows an else

If the part being replaced is an `if` expression following an `else`,
the replacement expression must be blockified.
@rustbot
Copy link
Collaborator

rustbot commented Feb 17, 2025

r? @Manishearth

rustbot has assigned @Manishearth.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 17, 2025
@Manishearth Manishearth added this pull request to the merge queue Feb 17, 2025
Merged via the queue into rust-lang:master with commit 5e4d166 Feb 17, 2025
11 checks passed
@samueltardieu samueltardieu deleted the push-nnvrxprxnspp branch February 18, 2025 11:52
# 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.

manual_ok_err automatic fix doesn't add {} when after else
3 participants