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

Add support for PHP 8.4 #191

Draft
wants to merge 2 commits into
base: 3.4.x
Choose a base branch
from

Conversation

cedric-anne
Copy link

Q A
Documentation no
Bugfix no
BC Break no
New Feature yes
RFC no
QA no

@cedric-anne
Copy link
Author

There are the following deprecations triggered by any PHP command in the CI env itself:

PHP Deprecated:  session.sid_length INI setting is deprecated in Unknown on line 0
PHP Deprecated:  session.sid_bits_per_character INI setting is deprecated in Unknown on line 0

For instance, they are triggered inside the CI init process itself:
image

I do not know how to fix them, and it makes the PHPUnit test failing.

@gsteel
Copy link
Member

gsteel commented Sep 4, 2024

An upgrade to PHPUnit 10 is probably required - see #190

@cedric-anne
Copy link
Author

An upgrade to PHPUnit 10 is probably required - see #190

Honestly, I thought it would take me only a few minutes at first. I proposed this to be able to activate PHP 8.4 checks on the CI of a dependency of a dependency we use. Unfortunately, I will not have time to investigate more on this right now.

Signed-off-by: Cédric Anne <cedric.anne@gmail.com>
@Xerkus Xerkus removed this from the 3.4.0 milestone Sep 10, 2024
Signed-off-by: Cédric Anne <cedric.anne@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

4 participants