Skip to content

Fix broken phar tests#743

Merged
mrsdizzie merged 4 commits intomainfrom
fix/update-versions-in-tests
Apr 30, 2025
Merged

Fix broken phar tests#743
mrsdizzie merged 4 commits intomainfrom
fix/update-versions-in-tests

Conversation

@swissspidy
Copy link
Copy Markdown
Member

@swissspidy swissspidy commented Apr 30, 2025

I noticed some failures in https://github.com/wp-cli/automated-tests/actions/runs/14743840901

They appear to happen because nowadays justinrainbow/json-schema depends on marc-mabe/php-enum, so this PR fixes that by adding this package to the phar file.

The tests here in this repo were not affected. Only the automated-tests repo.


Additionally, I took this as a opportunity to update the versions of WP-CLI used in tests to ensure they continue working fine on newer PHP versions. Previously: #658

@swissspidy swissspidy added the scope:testing Related to testing label Apr 30, 2025
@swissspidy swissspidy added this to the 2.12.0 milestone Apr 30, 2025
@swissspidy swissspidy requested a review from a team as a code owner April 30, 2025 09:19
@mrsdizzie mrsdizzie merged commit 03f3a41 into main Apr 30, 2025
42 of 43 checks passed
@mrsdizzie mrsdizzie deleted the fix/update-versions-in-tests branch April 30, 2025 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants