Skip to content

Conversation

rvanvelzen
Copy link
Contributor

Fix phpstan/phpstan#10715. Alternative to #3411 to handle conditional types correctly.

@ondrejmirtes ondrejmirtes merged commit a50b75a into phpstan:1.12.x Sep 9, 2024
487 of 500 checks passed
@ondrejmirtes
Copy link
Member

Thank you!

@VincentLanglet
Copy link
Contributor

Hi, this fix is definitely better since it seems to fix more issues than mine.

I feel like

Am I wrong ?

Do you plan to write non-regression test about these or do you want me to do it @rvanvelzen ?

@rvanvelzen
Copy link
Contributor Author

@VincentLanglet those issues seem to be solved as well indeed. Feel free to write tests for them 👍

@VincentLanglet
Copy link
Contributor

@VincentLanglet those issues seem to be solved as well indeed. Feel free to write tests for them 👍

Sure I'm on it.

To close phpstan/phpstan#9224 there is still an issue to solve.
Maybe you'll know why we have a different behavior (and how to improve your fix) in https://phpstan.org/r/e283bf82-214a-4115-a3b7-2c34b1d8b2cf

@rvanvelzen
Copy link
Contributor Author

@VincentLanglet the final issue there was in fact something completely different. I created #3433 to resolve it.

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.

3 participants