Skip to content

More precise ignore_user_abort() return type#2489

Merged
ondrejmirtes merged 1 commit intophpstan:1.10.xfrom
staabm:ignore_user_abort
Jun 26, 2023
Merged

More precise ignore_user_abort() return type#2489
ondrejmirtes merged 1 commit intophpstan:1.10.xfrom
staabm:ignore_user_abort

Conversation

@staabm
Copy link
Contributor

@staabm staabm commented Jun 26, 2023

as we recently stumbled over this functions return type I figured it would be helpful to cover this more precisely in PHPStan.

The setter expects a bool, but the getter returns a int.

https://3v4l.org/jmCa8
https://www.php.net/manual/en/function.ignore-user-abort.php

@staabm staabm changed the base branch from 1.11.x to 1.10.x June 26, 2023 09:27
@phpstan-bot
Copy link
Collaborator

You've opened the pull request against the latest branch 1.11.x. If your code is relevant on 1.10.x and you want it to be released sooner, please rebase your pull request and change its target to 1.10.x.

@staabm staabm force-pushed the ignore_user_abort branch from 15927c0 to 34636c4 Compare June 26, 2023 20:22
@ondrejmirtes ondrejmirtes merged commit 8937f3e into phpstan:1.10.x Jun 26, 2023
@ondrejmirtes
Copy link
Member

Thank you!

@staabm staabm deleted the ignore_user_abort branch June 27, 2023 03:38
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