Skip to content

Fix mt_rand return type #1956

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

Merged
merged 1 commit into from
Nov 4, 2022
Merged

Fix mt_rand return type #1956

merged 1 commit into from
Nov 4, 2022

Conversation

enumag
Copy link
Contributor

@enumag enumag commented Nov 3, 2022

Copy link
Member

@ondrejmirtes ondrejmirtes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What did you do to the icon? 👓

@enumag
Copy link
Contributor Author

enumag commented Nov 3, 2022

Huh? Nothing at all. I didn't touch it.

@enumag
Copy link
Contributor Author

enumag commented Nov 3, 2022

It must have somehow been deleted by git automatically because I have .idea in global gitignore... I'll add it back...

@ondrejmirtes
Copy link
Member

You deleted the file.

@enumag
Copy link
Contributor Author

enumag commented Nov 3, 2022

Fixed.

@enumag enumag force-pushed the mt-rand branch 2 times, most recently from d9a3dfa to db12e0d Compare November 4, 2022 07:43
@enumag enumag requested review from canvural and staabm and removed request for ondrejmirtes, canvural and staabm November 4, 2022 07:43
@enumag enumag requested review from ondrejmirtes and staabm and removed request for ondrejmirtes and staabm November 4, 2022 07:44
@ondrejmirtes
Copy link
Member

I can't review it when the tests are failing.

Verified

This commit was signed with the committer’s verified signature.
enumag Jáchym Toušek
@enumag
Copy link
Contributor Author

enumag commented Nov 4, 2022

@ondrejmirtes Fixed. There is still one failed job but I don't think it's related to my changes: https://github.com/phpstan/phpstan-src/actions/runs/3393592592/jobs/5641134975

@ondrejmirtes ondrejmirtes merged commit e3055ae into phpstan:1.9.x Nov 4, 2022
@ondrejmirtes
Copy link
Member

Thank you.

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.

mt_rand should have more specific return type
4 participants