Skip to content

Conversation

nikic
Copy link
Member

@nikic nikic commented Aug 9, 2025

They may be deprecated, but they should still produce the correct type...

They may be deprecated, but they should still produce the correct
type...
Copy link
Member

@nielsdos nielsdos left a comment

Choose a reason for hiding this comment

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

Welp, thanks for catching

@nikic nikic merged commit bc475ad into php:master Aug 9, 2025
9 checks passed
@Girgias
Copy link
Member

Girgias commented Aug 9, 2025

Yeah, that was a colossal brain fart.

@jrfnl
Copy link
Contributor

jrfnl commented Aug 9, 2025

Thanks for fixing this, I was already breaking my brain trying to figure out why the tokenization had changed (test in PHP_CodeSniffer which was failing).

@jrfnl
Copy link
Contributor

jrfnl commented Aug 9, 2025

For the record - this fixes an issue introduced via #19372.

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.

4 participants