Skip to content

fix(needless_if): don't expand macro invocations in the suggestion #15960

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.

Sign up for GitHub

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

Jump to bottom
Merged
samueltardieu merged 1 commit into from
Oct 27, 2025
Merged

fix(needless_if): don't expand macro invocations in the suggestion #15960

samueltardieu merged 1 commit into from
Oct 27, 2025

Conversation

@ada4a
Copy link
Contributor

@ada4a ada4a commented Oct 26, 2025

Fixes #15959

changelog: [needless_if]: don't expand macro invocations in the suggestion
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 26, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 26, 2025

r? @dswij

rustbot has assigned @dswij.
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
@ada4a ada4a mentioned this pull request Oct 26, 2025
Merged
@samueltardieu samueltardieu added this pull request to the merge queue Oct 27, 2025
Merged via the queue into rust-lang:master with commit a0a347e Oct 27, 2025
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 27, 2025
@ada4a ada4a deleted the needless_if branch October 27, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants

@ada4a @rustbot @samueltardieu @dswij