Skip to content

Int and float toString() produces numeric-string#371

Merged
ondrejmirtes merged 2 commits intophpstan:masterfrom
b1rdex:cast-to-numeric-string
Nov 11, 2020
Merged

Int and float toString() produces numeric-string#371
ondrejmirtes merged 2 commits intophpstan:masterfrom
b1rdex:cast-to-numeric-string

Conversation

@b1rdex
Copy link
Contributor

@b1rdex b1rdex commented Nov 11, 2020

This is for phpstan/phpstan#4005

@ondrejmirtes
Copy link
Member

Hi, this needs a regression test. Probably add a new dataProvider in NodeScopeResolverTest::testFileAsserts() and test that (string) $int and (string) $float return string&numeric.

@b1rdex b1rdex force-pushed the cast-to-numeric-string branch from 3c10d55 to 539d006 Compare November 11, 2020 14:58
@b1rdex
Copy link
Contributor Author

b1rdex commented Nov 11, 2020

@ondrejmirtes done

@b1rdex b1rdex force-pushed the cast-to-numeric-string branch from 7e3f1e3 to 65e7721 Compare November 11, 2020 15:34
@b1rdex
Copy link
Contributor Author

b1rdex commented Nov 11, 2020

@ondrejmirtes fixed

@ondrejmirtes ondrejmirtes merged commit 248a3ee into phpstan:master Nov 11, 2020
@ondrejmirtes
Copy link
Member

Thank you!

@b1rdex b1rdex deleted the cast-to-numeric-string branch November 11, 2020 15:53
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