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

Allow PHP 8.4 in laminas-cache-storage-adapter-redis 2.x #135

Merged
merged 4 commits into from
Jan 23, 2025

Conversation

driehle
Copy link
Contributor

@driehle driehle commented Jan 22, 2025

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

Description

This MR falls in the context of adding support for PHP 8.4 to laminas-cache v3. Adding support to the legacy version is essential, as laminas-cache v4 requires servicemanager v4 and many users cannot yet migrate to servicemanager v4, as quite a few packages in the MVC ecosystem yet lack support for servicemanager v4.

More information available here:

Signed-off-by: Dennis Riehle <[email protected]>
@driehle driehle changed the base branch from 3.2.x to 2.10.x January 22, 2025 12:19
@driehle
Copy link
Contributor Author

driehle commented Jan 22, 2025

@froschdesign Any idea why CI is not running here? Does it need approval from your side?

Edit: Nevermind. The second push triggered CI. Not sure why the first push didn't trigger CI though...

@driehle
Copy link
Contributor Author

driehle commented Jan 22, 2025

make failes with ext-redis 5.3.7 on PHP 8.4. Could it be that 5.3.7 is not compatible with PHP 8.4?

Edit: This seems indeed to be the case. The release notes of phpredis 6.1.0 mention the following:

@driehle driehle force-pushed the feat/php84 branch 3 times, most recently from 789c034 to 1a9693b Compare January 22, 2025 12:49
@driehle driehle force-pushed the feat/php84 branch 2 times, most recently from 30534bd to 12f3df7 Compare January 22, 2025 13:03
Signed-off-by: Dennis Riehle <[email protected]>
Copy link
Member

@gsteel gsteel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @driehle

@gsteel gsteel self-assigned this Jan 23, 2025
@gsteel gsteel merged commit ead8443 into laminas:2.10.x Jan 23, 2025
24 checks passed
@driehle driehle deleted the feat/php84 branch January 23, 2025 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants