-
Notifications
You must be signed in to change notification settings - Fork 19
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
upkeep/183: bump min WP and PHP versions #197
Conversation
@Sidsector9 thanks for the PR! Could you please fill out the PR template with description, changelog, and credits information so that we can properly review and merge this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @Sidsector9, changes look good to me. Noticed one minor thing, I think we should update WP minimum version to 5.7 in E2E testing GH Action here as well.
Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes @Sidsector9, LGTM 🚀
@jeffpaul could you please help with removing the required
checks PHP minimum 7.2
and phpunit (7.3)
which are no longer needed? to unblock merging here.
Thanks
Description of the Change
This PR bumps minimum WP and PHP versions.
Closes #183
Changelog Entry
Credits
Props @vikrampm1 @Sidsector9
Checklist: