Skip to content
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

ci: Fix & add PHP 8.4 #92

Merged
merged 4 commits into from
Oct 10, 2024
Merged

ci: Fix & add PHP 8.4 #92

merged 4 commits into from
Oct 10, 2024

Conversation

jtojnar
Copy link
Contributor

@jtojnar jtojnar commented Oct 9, 2024

PHP 8.4 is in beta, with final version scheduled for November so it is time to start testing it.

With php-cs-fixer 3.64.0, the `native_function_invocation` rule no longer passed.
It will be deprecated in PHP 8.4 and it is meaningless nowadays anyway:
https://wiki.php.net/rfc/deprecations_php_8_4#remove_e_strict_error_level_and_deprecate_e_strict_constant

The use of the constant was introduced in 175196d.
PHP 8.4 is in beta, with final version scheduled for November so it is time to start testing it.
It is unused and would cause an error on PHP ≥ 8.0:
https://www.php.net/manual/en/function.set-error-handler.php#refsect1-function.set-error-handler-parameters

Not sure if the handler is even necessary – it was introduced in 175196d but I did not manage to reproduce the original error (Entity 'nbsp' not defined). It was probably fixed by f2a43b4.
@j0k3r j0k3r merged commit 6947999 into j0k3r:master Oct 10, 2024
10 checks passed
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