Closed bonfante closed 5 years ago
Thanks for the pull request @bonfante! Please see my comments.
This pull request is featuring breaking changes and will require a new major release.
@ivanakimov any thoughts?
@bonfante Thanks for the PR! @vinkla Thanks for the quick follow-up -- I am following your lead on this one!
@vinkla done 👍.
Thanks @bonfante! ✨
We'll bump the branch alias to 4.0, please try using the new version in any project. We'll release a new version in a week or two.
mb_ord()
could be polyfilled thus no need to be PHP72-only actually.
https://github.com/symfony/polyfill-php72/blob/master/Php72.php#L191
That is great! We’ll probably leave it as it is since 7.1 will reach EOL this year anyway.
UTF-8 support.
Extension mbstring become required and php ^7.2 to use mb_ord