Skip to content

Update tests to avoid deprecated use of the Reflection API - #168

Merged
clue merged 1 commit into
ratchetphp:masterfrom
mbabker:fix-reflection-deprecations
Jul 28, 2026
Merged

Update tests to avoid deprecated use of the Reflection API#168
clue merged 1 commit into
ratchetphp:masterfrom
mbabker:fix-reflection-deprecations

Conversation

@mbabker

@mbabker mbabker commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

PHP 8.1 effectively made the various reflection classes setAccessible methods a no-op, with them being formally deprecated in PHP 8.5. This PR silences those deprecations (found while testing support for guzzlehttp/psr7:^3.0 but blocked by ratchetphp/RFC6455#79).

@clue clue added this to the 0.4.4 milestone Jul 28, 2026
@clue clue changed the title Avoid deprecated use of the Reflection API Update tests to avoid deprecated use of the Reflection API Jul 28, 2026

@clue clue left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@mbabker Thanks for the update, changes LGTM! :shipit:

@clue
clue merged commit f9f0e4c into ratchetphp:master Jul 28, 2026
6 checks passed
@mbabker
mbabker deleted the fix-reflection-deprecations branch July 28, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants