Skip to content

Do not trigger let_and_return on let else#16829

Merged
samueltardieu merged 1 commit intorust-lang:masterfrom
samueltardieu:issues/16820
Apr 16, 2026
Merged

Do not trigger let_and_return on let else#16829
samueltardieu merged 1 commit intorust-lang:masterfrom
samueltardieu:issues/16820

Conversation

@samueltardieu
Copy link
Copy Markdown
Member

@samueltardieu samueltardieu commented Apr 9, 2026

changelog: [let_and_return_on]: do not trigger on let else construct

Fixes #16820

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 9, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 9, 2026

r? @Jarcho

rustbot has assigned @Jarcho.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: 7 candidates
  • 7 candidates expanded to 7 candidates
  • Random selection from Jarcho, dswij, llogiq

@rustbot

This comment has been minimized.

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 11, 2026

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

Copy link
Copy Markdown
Contributor

@Jarcho Jarcho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Jarcho Jarcho added this pull request to the merge queue Apr 16, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 16, 2026
@samueltardieu
Copy link
Copy Markdown
Member Author

Spurious error, reenqueuing.

@samueltardieu samueltardieu added this pull request to the merge queue Apr 16, 2026
Merged via the queue into rust-lang:master with commit 121f867 Apr 16, 2026
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 16, 2026
@samueltardieu samueltardieu deleted the issues/16820 branch April 16, 2026 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

False positive for let_and_return

3 participants