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

Bumped WP and PHP Minimums #80

Merged
merged 9 commits into from
Jan 5, 2023
Merged

Conversation

zamanq
Copy link
Contributor

@zamanq zamanq commented Oct 7, 2022

Description of the Change

Bumped up WordPress PHP minimums.
Closes #73

Changelog Entry

Added - Bumped up WordPress and PHP Minimum Requirements

Credits

Props @jeffpaul @zamanq

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@zamanq zamanq requested a review from jeffpaul October 7, 2022 17:28
@jeffpaul jeffpaul added this to the 1.2.0 milestone Oct 7, 2022
@jeffpaul jeffpaul requested review from a team and faisal-alvi and removed request for jeffpaul and a team October 7, 2022 17:34
faisal-alvi
faisal-alvi previously approved these changes Oct 10, 2022
@faisal-alvi
Copy link
Member

@zamanq I've approved the PR but I've just noticed that the phpunit and phpcs tests are failing. Can you please check them?

@jeffpaul jeffpaul added the help wanted Extra attention is needed label Oct 27, 2022
@jeffpaul
Copy link
Member

Noting that if anyone is able to help with the test failures here that would help move this PR along, thanks!

@mehul0810
Copy link
Contributor

@jeffpaul I have resolved the merge conflicts and updated code to ensure the tests doesn't fail due to the update in PHP version. Also, to mention that the PHPCS tests are failing due to a known bug (WordPress/WordPress-Coding-Standards#2035) in WPCS. So, I have added a temporary work around to that until they release a new version of WPCS.

@jeffpaul jeffpaul merged commit edce75c into develop Jan 5, 2023
@jeffpaul jeffpaul deleted the add/73-bump-up-wp-php-minimums branch January 5, 2023 14:46
@jeffpaul jeffpaul modified the milestones: 1.2.0, 1.1.1 Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bump WordPress and PHP minimums
4 participants