Skip to content

[CodeQuality] Handle with assign on SimplifyIfElseToTernaryRector - #7951

Merged
TomasVotruba merged 2 commits into
mainfrom
with-assign
Apr 4, 2026
Merged

TomasVotruba merged 2 commits into
mainfrom
with-assign

Conversation

@samsonasik

Copy link
Copy Markdown
Member

@samsonasik
samsonasik requested a review from TomasVotruba April 4, 2026 03:17
@samsonasik

Copy link
Copy Markdown
Member Author

Fixed 🎉 /cc @sreichel

@samsonasik

Copy link
Copy Markdown
Member Author

@TomasVotruba ready 👍

@sreichel

sreichel commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

Thanks. ❤️

@sreichel

sreichel commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

I am highly impressed by your knowledge/routine.

Changing 3 lines .... done.

@TomasVotruba
TomasVotruba merged commit 9dccc8a into main Apr 4, 2026
60 checks passed
@TomasVotruba
TomasVotruba deleted the with-assign branch April 4, 2026 07:34
@TomasVotruba

Copy link
Copy Markdown
Member

Thank you 👍

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect behavior of SimplifyIfElseToTernaryRector

4 participants