Skip to content

[Php81] Skip with next required on NewInInitializerRector#6599

Merged
samsonasik merged 5 commits intomainfrom
skip-with-next-required
Dec 17, 2024
Merged

[Php81] Skip with next required on NewInInitializerRector#6599
samsonasik merged 5 commits intomainfrom
skip-with-next-required

Conversation

@samsonasik
Copy link
Member

@samsonasik samsonasik commented Dec 17, 2024

@samsonasik
Copy link
Member Author

Test notice on ReturnTypeFromStrictNewArrayRector is due to new phpstan 2.0.4 https://github.com/phpstan/phpstan/releases/tag/2.0.4

I updated the fixture 977c71e

@samsonasik
Copy link
Member Author

Fixed 🎉 /cc @NicoImparfait

@samsonasik
Copy link
Member Author

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

@samsonasik samsonasik merged commit e1b032e into main Dec 17, 2024
41 checks passed
@samsonasik samsonasik deleted the skip-with-next-required branch December 17, 2024 21:54
@NicoImparfait
Copy link
Contributor

@samsonasik thank you !

@github-actions
Copy link
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 Jan 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect behavior of NewInInitializerRector with required parameters

3 participants