Skip to content

Conversation

@biozshock
Copy link
Contributor

Test to identify the issue.

As far as i was able to debug the issue is actually in the doc parsing in the phpstan.
In this case the $variances here is an empty array.
Though the PHPStan itself processes the "before" and "after" correctly.

@samsonasik
Copy link
Member

could you fix rectify:

composer update
vendor/bin/rector && composer fix-cs

@biozshock
Copy link
Contributor Author

There's a rector change in the other part of the code. Should it be fixed here, or in a separate PR?

@samsonasik
Copy link
Member

Run here seems fine as only single file

@samsonasik
Copy link
Member

Test seems green

@biozshock
Copy link
Contributor Author

Well, yes, but you can check the phpunit output :).
Added an option for phpunit to fail on warnings and notices.

@samsonasik
Copy link
Member

@samsonasik
Copy link
Member

I pushed empty commit to trigger latest rector-src 55220b8 👍

@samsonasik
Copy link
Member

@biozshock All green 🎉 @TomasVotruba I am merging it ;)

@samsonasik samsonasik merged commit 8ce26ae into rectorphp:main Aug 8, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants