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

upkeep/40: PHP8 compat workflow #39

Merged
merged 5 commits into from
Apr 6, 2022
Merged

upkeep/40: PHP8 compat workflow #39

merged 5 commits into from
Apr 6, 2022

Conversation

Sidsector9
Copy link
Member

@Sidsector9 Sidsector9 commented Mar 23, 2022

Description of the Change

  • Added PHP8 Compatibility workflow
  • Bumped minimum WP version to 5.8
  • Bumped minimum PHP version to 5.6

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests passed.

Changelog Entry

upkeep: Added PHP8 compatibility workflow

Credits

Props @Sidsector9

@Sidsector9 Sidsector9 self-assigned this Mar 23, 2022
@Sidsector9 Sidsector9 added this to the 1.1.0 milestone Mar 23, 2022
@Sidsector9 Sidsector9 requested review from a team and faisal-alvi and removed request for a team March 23, 2022 20:26
@Sidsector9 Sidsector9 changed the title Php8 compat upkeep/40: Php8 compat workflow Mar 23, 2022
faisal-alvi
faisal-alvi previously approved these changes Apr 1, 2022
Copy link
Member

@faisal-alvi faisal-alvi left a comment

Choose a reason for hiding this comment

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

@Sidsector9 LGTM - just requires a version bump before the merge. Approving now, as the change is minor.

* Requires at least: 5.2
* Requires at least: 5.8
Copy link
Member

Choose a reason for hiding this comment

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

The version should also be bumped in readme files.

Requires at least: 5.2

* [WordPress](http://wordpress.org/) 5.2+

@Sidsector9 Sidsector9 changed the title upkeep/40: Php8 compat workflow upkeep/40: PHP8 compat workflow Apr 2, 2022
@jeffpaul jeffpaul merged commit 8f6be2c into develop Apr 6, 2022
@jeffpaul jeffpaul deleted the php8-compat branch April 6, 2022 14:45
@jeffpaul jeffpaul linked an issue Apr 12, 2022 that may be closed by this pull request
3 tasks
@@ -4,7 +4,7 @@
* Plugin URI: https://wordpress.org/plugins/retro-winamp-block/
* Description: A Winamp-styled audio block for all your retro music player needs.
* Version: 1.0.1
* Requires at least: 5.2
* Requires at least: 5.8
* Requires PHP: 7.2
Copy link
Member

Choose a reason for hiding this comment

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

Should this get changed to 5.6?

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.

Add PHP8 Compatibility testing workflow
3 participants