Skip to content

Update test.yml#665

Merged
kasparsd merged 1 commit intoWordPress:masterfrom
BrookeDot:patch-2
Mar 14, 2025
Merged

Update test.yml#665
kasparsd merged 1 commit intoWordPress:masterfrom
BrookeDot:patch-2

Conversation

@BrookeDot
Copy link
Copy Markdown
Contributor

What?

This PR updates the tests to:

  • Add PHP 8.4
  • Update builds to 8.3

Why?

This should help ensure that the plugin works well with PHP 8.4 moving forward and ensure that the tests work better on a slightly updated PHP version 8.3.

How?

Manually edits the test file.

Changelog Entry

Changed - Update tests to include PHP 8.4

- Adds PHP 8.4
- Updates builds to 8.3
Copy link
Copy Markdown
Collaborator

@kasparsd kasparsd left a comment

Choose a reason for hiding this comment

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

This is great, thank you!

@kasparsd
Copy link
Copy Markdown
Collaborator

Error while running docker compose command.
failed to solve: wordpress:php8.4: failed to resolve source metadata for docker.io/library/wordpress:php8.4: docker.io/library/wordpress:php8.4: not found

Looks like there is a known issue with missing 8.4 docker-library/wordpress#932

@BrookeDot
Copy link
Copy Markdown
Contributor Author

Closing this for now as the WordPress docker doesn't support 8.4 yet, so we'll have to wait or switch repos.

@BrookeDot BrookeDot closed this Feb 13, 2025
@BrookeDot
Copy link
Copy Markdown
Contributor Author

I'll follow that issue and re-open once PHP 8.4 support is available.

@BrookeDot
Copy link
Copy Markdown
Contributor Author

Re-opening as PHP 8.4 is now supported by the WordPress Docker image used by the tests.

@BrookeDot BrookeDot reopened this Mar 14, 2025
@jeffpaul jeffpaul added this to the 0.13.0 milestone Mar 14, 2025
@jeffpaul jeffpaul requested a review from kasparsd March 14, 2025 16:13
@BrookeDot
Copy link
Copy Markdown
Contributor Author

Yay! All tests passed!

Copy link
Copy Markdown
Collaborator

@kasparsd kasparsd left a comment

Choose a reason for hiding this comment

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

This is great, thank you!

@kasparsd kasparsd merged commit e4df9c4 into WordPress:master Mar 14, 2025
30 of 54 checks passed
@kasparsd kasparsd mentioned this pull request Apr 2, 2025
@jeffpaul jeffpaul modified the milestones: 0.14.0, 0.13.0 Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants