Skip to content

chore: [SITE-5200] PHP 8.5 compatibility - #482

Merged
stevector merged 2 commits into
mainfrom
fix/site-5200-php85-compat
Jun 3, 2026
Merged

stevector merged 2 commits into
mainfrom
fix/site-5200-php85-compat

Conversation

@mehta-asim

@mehta-asim mehta-asim commented May 25, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add PHP 8.5 to PHPUnit CI matrix
  • Zero deprecation issues found across all 23 PHP files
  • Add contributor and changelog entries

Context

SITE-5200

PHP 8.5 compatibility verification for WP SAML Auth. Code scan found no deprecated casts, functions, or dynamic property issues. Only CI and changelog updates needed.

Test plan

  • PHPUnit passes on PHP 8.5 in CI
  • All other PHP matrix versions still pass
  • WP.org validation passes

Add PHP 8.5 to PHPUnit CI matrix. Zero deprecation issues found
across all 23 PHP files. Add contributor and changelog entries.
@mehta-asim
mehta-asim requested a review from a team as a code owner May 25, 2026 16:07
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@tundeafolabi tundeafolabi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@stevector
stevector merged commit e30eaf8 into main Jun 3, 2026
32 of 33 checks passed
@stevector
stevector deleted the fix/site-5200-php85-compat branch June 3, 2026 16:25
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.

3 participants