-
-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PHP 8.2] Fix ${var}
string interpolation deprecation
#181
Conversation
PHP 8.2 deprecates `"${var}"` string interpolation pattern. This fixes all 8 of such occurrences in `voku/portable-utf8` package. - [PHP 8.2: `${var}` string interpolation deprecated](https://php.watch/versions/8.2/${var}-string-interpolation-deprecated) - [RFC](https://wiki.php.net/rfc/deprecate_dollar_brace_string_interpolation)
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
thank you @Ayesh 👍 I just set up a PHP 8.2 pipeline for our project and this was the only issue so far on our side and I was about to do the same as you 😁 |
Codecov Report
@@ Coverage Diff @@
## master #181 +/- ##
=========================================
Coverage 81.34% 81.34%
Complexity 1766 1766
=========================================
Files 2 2
Lines 3950 3950
=========================================
Hits 3213 3213
Misses 737 737
Continue to review full report at Codecov.
|
just came here to report the same thing ;-). |
@voku anything we can do so this PR can be merged? |
Thanks, @Ayesh and sorry for the delay. I will tag a new version. |
PHP 8.2 deprecates
"${var}"
string interpolation pattern.This fixes all 8 of such occurrences in
voku/portable-utf8
package.${var}
string interpolation deprecatedThis change is