Skip to content

RFC: Deprecate metaphone() - #21889

Merged
LamentXU123 merged 6 commits into
php:masterfrom
LamentXU123:RFC-deprecate-metaphone
Aug 10, 2026
Merged

RFC: Deprecate metaphone()#21889
LamentXU123 merged 6 commits into
php:masterfrom
LamentXU123:RFC-deprecate-metaphone

Conversation

@LamentXU123

@LamentXU123 LamentXU123 commented Apr 27, 2026

Copy link
Copy Markdown
Member

RFC: https://wiki.php.net/rfc/deprecations_php_8_6#deprecate_metaphone_function
Internal thread: https://news-web.php.net/php.internals/130691

no need to look at this until the RFC is accepted/rejected :)

@LamentXU123 LamentXU123 changed the title initial version 1.0 RFC: Deprecate metaphone functions Apr 27, 2026
@TimWolla TimWolla added the RFC label Apr 27, 2026
Comment thread UPGRADING Outdated
@LamentXU123 LamentXU123 changed the title RFC: Deprecate metaphone functions RFC: Deprecate metaphone() Aug 10, 2026
@LamentXU123
LamentXU123 force-pushed the RFC-deprecate-metaphone branch 2 times, most recently from ad59edb to c5e1a74 Compare August 10, 2026 12:21
@LamentXU123

LamentXU123 commented Aug 10, 2026

Copy link
Copy Markdown
Member Author

The RFC is now accepted. @devnexen Can you give a quick review ? :) as we are very very very close to the feature freeze.

@LamentXU123
LamentXU123 marked this pull request as ready for review August 10, 2026 14:42
Comment thread ext/standard/tests/strings/metaphone.phpt Outdated
Comment thread ext/standard/basic_functions.stub.php Outdated
@LamentXU123
LamentXU123 requested a review from devnexen August 10, 2026 17:07

@devnexen devnexen 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.

lgtm once CI is green

@LamentXU123
LamentXU123 force-pushed the RFC-deprecate-metaphone branch from 42f40b7 to 77769fb Compare August 10, 2026 18:09
@LamentXU123
LamentXU123 merged commit 4b26ff1 into php:master Aug 10, 2026
18 checks passed
@LamentXU123
LamentXU123 deleted the RFC-deprecate-metaphone branch August 10, 2026 19:02
@LamentXU123

Copy link
Copy Markdown
Member Author

Thanks!

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.

6 participants