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

Increase minimum WordPress version to 5.9 #1035

Merged
merged 4 commits into from
Apr 28, 2024

Conversation

alecgeatches
Copy link
Contributor

Description

Increase the minimum supported WordPress version to 5.9, which allows for a simpler tests setup, since all of the PHPUnit-Polyfills are bundled with WordPress core. This includes a handful of changes:

  • Increase the plugin's minimum WordPress version to 5.9
  • Update integration tests to run against WordPress 5.9 at minimum
  • Remove unnecessary composer libraries needed for < 5.9

@alecgeatches alecgeatches self-assigned this Apr 22, 2024
@alecgeatches alecgeatches changed the title Increase minimum wordpress version to 5.9 Increase minimum WordPress version to 5.9 Apr 22, 2024
@alecgeatches alecgeatches marked this pull request as ready for review April 22, 2024 21:49
@alecgeatches
Copy link
Contributor Author

alecgeatches commented Apr 22, 2024

These tests are failing due to an older PHP version and mismatched dependency versions. After #1034 is merged, the tests here should pass. Please review that one first, thank you!

@GaryJones
Copy link
Contributor

After #1034 is merged, I'll want to rebase this and add some extra changes that would otherwise create merge conflicts.

@GaryJones GaryJones force-pushed the fix/minimum-wordpress-version branch from fb7e41e to a6f2bb5 Compare April 26, 2024 08:53
@GaryJones
Copy link
Contributor

Rebased on to develop, and tweaked the integration test dependency ranges.

Doesn't make sense to test WP versions would unsupported PHP versions (e.g. WP 5.9 with PHP 8.3).
@alecgeatches
Copy link
Contributor Author

✅ Looks good to me!

@GaryJones GaryJones merged commit 3ddc38c into develop Apr 28, 2024
7 checks passed
@GaryJones GaryJones deleted the fix/minimum-wordpress-version branch April 28, 2024 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants