-
Notifications
You must be signed in to change notification settings - Fork 665
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
PHP 8.4 support #11107
Comments
Hey @kkmuffme, can you reproduce the issue on https://psalm.dev? These will be used as phpunit tests when implementing the feature or fixing this bug. |
Psalm doesn't support PHP 8.4 yet, see vimeo/psalm#11107
Psalm error handler (https://github.com/vimeo/psalm/blob/5.26.1/src/Psalm/Internal/ErrorHandler.php#L62) do not allow to disable deprecations at runtime ( |
"This version of Nextcloud is not compatible with PHP>=8.4. Is there a distinction between 8.4 and 8.4.1 ? |
* Fix vimeo#11189 * Fix PHP_DEBUG constant for PHP 8.4 part of vimeo#11107
* Fix vimeo#11189 * Fix PHP_DEBUG constant for PHP 8.4 part of vimeo#11107
* Fix vimeo#11189 * Fix PHP_DEBUG constant for PHP 8.4 part of vimeo#11107
https://wiki.php.net/rfc/deprecations_php_8_4
https://github.com/php/php-src/blob/php-8.4.0RC1/UPGRADING#L102 (e.g.
The type of PHP_DEBUG and PHP_ZTS constants changed to bool.
)update ProjectAnalyzer
private const PHP_SUPPORTED_VERSIONS_REGEX = '^(5\.[456]|7\.[01234]|8\.[0123])(\..*)?$';
update composer.json
all changes done for 8.2/8.3 in PR added GitHub CI for unit tests with PHP 8.2 and 8.3 #10028 need to be done for 8.4 too
The text was updated successfully, but these errors were encountered: