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

Fix PHP 8.3 deprecations #2836

Merged
merged 1 commit into from
Jul 23, 2024
Merged

Fix PHP 8.3 deprecations #2836

merged 1 commit into from
Jul 23, 2024

Conversation

yscik
Copy link
Contributor

@yscik yscik commented Jul 22, 2024

Fixes #2807

Changes Proposed in this Pull Request

  • Fix some code deprecations

Testing Instructions

  • No logic changes

Release Notes

  • Fix PHP 8.3 support

Plugin build for 72f6d73
📦 Download plugin zip
▶️ Open in playground

@yscik yscik requested a review from a team July 22, 2024 15:26
@yscik yscik added this to the 2.4.0 milestone Jul 22, 2024
Copy link
Contributor

@pgk pgk left a comment

Choose a reason for hiding this comment

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

@yscik Looks good, wondering, should we add php 8.3 on .github/workflows/php.yml and other places where different php versions are used?

@yscik
Copy link
Contributor Author

yscik commented Jul 23, 2024

Looks like we have 7.4 and 8.3 for unit tests, that should be enough.

The linter should be checking for compatibility with PHP 7.4 to [latest] via PHPCompatibilty, but unfortunately it's not being run for a lot of the older code that doesn't meet the coding standards.

@yscik yscik merged commit f0e7208 into trunk Jul 23, 2024
12 checks passed
@yscik yscik deleted the fix/deprecated-php83 branch July 23, 2024 15:38
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.

PHP 8.3 deprecated notice for class-wp-job-manager-usage-tracking.php
2 participants