Skip to content

Bumped Up WP and PHP Minimums#96

Merged
jeffpaul merged 1 commit intodevelopfrom
add/93-bump-wp-php-minimums
Nov 18, 2022
Merged

Bumped Up WP and PHP Minimums#96
jeffpaul merged 1 commit intodevelopfrom
add/93-bump-wp-php-minimums

Conversation

@zamanq
Copy link
Contributor

@zamanq zamanq commented Oct 7, 2022

Description of the Change

Bumped up WP and PHP minimums. Also having some issues with the README.MD file formatting while saving so skipped that file to update the version numbers.
Closes #93

Changelog Entry

Added - Bumped up WordPress and PHP Minimums

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 19:17
@zamanq zamanq self-assigned this Oct 7, 2022
@jeffpaul jeffpaul added this to the 1.3.0 milestone Oct 7, 2022
@jeffpaul jeffpaul requested review from a team and cadic and removed request for a team and jeffpaul October 7, 2022 19:27
Copy link

@cadic cadic left a comment

Choose a reason for hiding this comment

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

@zamanq thanks for this update. I am concerned about using 7.4 here instead of 7.0 which looks a better option to me.

"minimum-stability": "dev",
"require": {
"php": ">=5.4"
"php": ">=7.4"
Copy link

Choose a reason for hiding this comment

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

I wonder is 7.4 a real requirement within the codebase of this plugin? 7.0-7.3 still take more than 20% of setups according to WordPress Stats

Copy link
Member

Choose a reason for hiding this comment

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

@cadic I imagine 7.4 was taken from the linked issue #93 which is what we want to bump our plugin minimums to at this point

Copy link
Member

Choose a reason for hiding this comment

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

@cadic that sound ok to you or should we stick with 7.0?

@jeffpaul jeffpaul merged commit 47492ed into develop Nov 18, 2022
@jeffpaul jeffpaul deleted the add/93-bump-wp-php-minimums branch November 18, 2022 17:47
@dkotter dkotter modified the milestones: 1.3.0, 1.2.1 Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bump WordPress and PHP minimums

4 participants