Skip to content

[Php82] Handle implicit "public" on construtor promotion on ReadOnlyClassRector#6895

Merged
samsonasik merged 3 commits intomainfrom
implicit
May 12, 2025
Merged

[Php82] Handle implicit "public" on construtor promotion on ReadOnlyClassRector#6895
samsonasik merged 3 commits intomainfrom
implicit

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@samsonasik
Copy link
Copy Markdown
Member Author

Fixed 🎉 /cc @andrew-demb

@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba I am merging it ;)

@samsonasik samsonasik merged commit c229e8e into main May 12, 2025
45 checks passed
@samsonasik samsonasik deleted the implicit branch May 12, 2025 13:49
@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 Nov 29, 2025
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 RemoveEmptyClassMethodRector, RemoveUnusedConstructorParamRector

1 participant