Skip to content

Fix dynamic return type extension for wp_die()#204

Merged
szepeviktor merged 3 commits intoszepeviktor:masterfrom
IanDelMar:wp-die
Oct 16, 2023
Merged

Fix dynamic return type extension for wp_die()#204
szepeviktor merged 3 commits intoszepeviktor:masterfrom
IanDelMar:wp-die

Conversation

@IanDelMar
Copy link
Copy Markdown
Contributor

@IanDelMar IanDelMar commented Oct 16, 2023

Fixes #203
Adds missing tests (#203 (comment))

@IanDelMar IanDelMar changed the title Add missing tests for wp_die() Fix dynamic return type extension for wp_die() Oct 16, 2023
@IanDelMar
Copy link
Copy Markdown
Contributor Author

@szepeviktor NonAcceptingNeverType requires PHPStan 1.10.30

@herndlm
Copy link
Copy Markdown
Contributor

herndlm commented Oct 16, 2023

Bump it Victor, bump it! :)

@szepeviktor szepeviktor merged commit b8516ed into szepeviktor:master Oct 16, 2023
@szepeviktor
Copy link
Copy Markdown
Owner

requires PHPStan 1.10.30

With 1.10.30 it fails to find NonAcceptingNeverType
https://app.travis-ci.com/github/szepeviktor/phpstan-wordpress/jobs/611654029

@szepeviktor
Copy link
Copy Markdown
Owner

It is https://github.com/phpstan/phpstan/releases/tag/1.10.31 :(

@IanDelMar IanDelMar deleted the wp-die branch October 16, 2023 17:48
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.

wp_die() doesn't end script anymore

3 participants