Skip to content

Conversation

@Soean
Copy link
Contributor

@Soean Soean commented Nov 28, 2025

Fixes #119

@Soean Soean marked this pull request as ready for review November 28, 2025 11:17
@Soean Soean changed the title Add PHP 8.5 support WIP: Add PHP 8.5 support Nov 28, 2025
@Soean Soean changed the title WIP: Add PHP 8.5 support Add PHP 8.5 support Nov 28, 2025
@Soean Soean marked this pull request as draft November 28, 2025 11:18
@Chrico Chrico requested a review from gmazzap November 28, 2025 12:49
@Soean Soean marked this pull request as ready for review November 28, 2025 12:54
@gmazzap
Copy link
Contributor

gmazzap commented Nov 28, 2025

@Soean It seems WordPress has not yet published the official Docker image for PHP 8.5.

So integration test fails for trying to download an image that does not exist.

Can you confirm the integration tests pass on PHP 8.5? If so, we can remove the tests for now and merge this.

@gmazzap
Copy link
Contributor

gmazzap commented Nov 28, 2025

We should also lock Patchwork dependency to the latest 2.2.3, or unit tests fail for "lowest" depndecies installation.

@Soean
Copy link
Contributor Author

Soean commented Dec 2, 2025

@gmazzap The docker image is released, the tests look good ✅

@gmazzap gmazzap merged commit 60b3a10 into inpsyde:3.x Dec 2, 2025
41 checks passed
@Soean Soean deleted the php_8_5 branch December 2, 2025 07:52
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.

[Feature Request]: PHP 8.5 Support

3 participants