Skip to content

[Arguments] Skip self class const fetch inside this class target replacement on ReplaceArgumentDefaultValueRector - #7940

Merged
TomasVotruba merged 7 commits into
mainfrom
skip-self
Mar 28, 2026
Merged

[Arguments] Skip self class const fetch inside this class target replacement on ReplaceArgumentDefaultValueRector#7940
TomasVotruba merged 7 commits into
mainfrom
skip-self

Conversation

@samsonasik

Copy link
Copy Markdown
Member

@samsonasik

Copy link
Copy Markdown
Member Author

Fixed 🎉 /cc @sreichel

@samsonasik
samsonasik requested a review from TomasVotruba March 28, 2026 07:49
@samsonasik

Copy link
Copy Markdown
Member Author

@TomasVotruba ready 👍

@samsonasik samsonasik changed the title [Arguments] Skip self class const fetch inside this class itself on ReplaceArgumentDefaultValueRector [Arguments] Skip self class const fetch inside this class target replacement on ReplaceArgumentDefaultValueRector Mar 28, 2026
@sreichel

Copy link
Copy Markdown
Contributor

Great. Thanks.

@TomasVotruba
TomasVotruba merged commit badcffc into main Mar 28, 2026
60 checks passed
@TomasVotruba
TomasVotruba deleted the skip-self branch March 28, 2026 14:09
@TomasVotruba

Copy link
Copy Markdown
Member

Thank you

@github-actions

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 Aug 30, 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.

Respect self when ranaming

4 participants