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 Up WP and PHP Minimums #96

Merged
merged 1 commit into from
Nov 18, 2022
Merged

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 jeffpaul and a team 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.

@@ -15,7 +15,7 @@
],
"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