-
Notifications
You must be signed in to change notification settings - Fork 545
IntegerType::toString is lowercase #3510
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
8f88c09 to
8a56599
Compare
I recently learned about https://3v4l.org/nvKP7
(削除) not sure thats something anyone is doing though (削除ここまで)
Update: ahh its a float, doesn't matter then: https://3v4l.org/sC13o
Yes,
Int to string is a lowercase string but not float to string.
328b6ad
into
phpstan:1.12.x
Thank you!
Hi, this broke tests on phpstan-doctrine for some PHP versions and I don't understand why only for some. I'm reverting this. Feel free to investigate and fix it there. Thanks.
See https://github.com/phpstan/phpstan/actions/runs/11202848434
Hi, this broke tests on phpstan-doctrine for some PHP versions and I don't understand why only for some. I'm reverting this. Feel free to investigate and fix it there. Thanks.
Hi @ondrejmirtes this is "expected", the behavior seems to be different in PHP < 8.1
cf this code
https://github.com/phpstan/phpstan-doctrine/blob/14a59bb5b719d3740224b6ba8aec80c0294142f9/tests/Type/Doctrine/Query/QueryResultTypeWalkerTest.php#L1633-L1643
I assume adding the LowerCase accessory to https://github.com/phpstan/phpstan-doctrine/blob/14a59bb5b719d3740224b6ba8aec80c0294142f9/tests/Type/Doctrine/Query/QueryResultTypeWalkerTest.php#L1590 might solve all the issues.
So to me, it's all good, we could revert the revert and just fix the tests.
Please try to send a PR to phpstan-doctrine on 1.5.x branch and pin phpstan/phpstan to the non-reverted version (1.12.6).
Please try to send a PR to phpstan-doctrine on 1.5.x branch and pin phpstan/phpstan to the non-reverted version (1.12.6).
Uh oh!
There was an error while loading. Please reload this page.
Which is useful to keep
as a lowercase string.