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.1 supporting #347
Comments
Any news? |
PR #346 was opened to address the first deprecation, but it hasn't been merged yet. @greenlion Any idea when this might get merged please? |
Will look at the PR this weekend and merge.
…On Fri, Jan 28, 2022, 12:46 PM Gareth Ellis ***@***.***> wrote:
PR #346 <#346> was opened
to address the first deprecation, but it hasn't been merged yet.
@greenlion <https://github.com/greenlion> Any idea when this might get
merged please?
—
Reply to this email directly, view it on GitHub
<#347 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADRTNQHFSCTP5HUVA6QQEDUYLI7BANCNFSM5LHZOS6Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Thank you. Would you like a PR for the second deprecation? |
Yes, please!
…On Fri, Jan 28, 2022, 12:51 PM Gareth Ellis ***@***.***> wrote:
Thank you. Would you like a PR for the second deprecation?
—
Reply to this email directly, view it on GitHub
<#347 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADRTNU6MNXPLWQG2URK5XTUYLJTBANCNFSM5LHZOS6Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
It doesn't look like it's possible to run the test suite on PHP 8.1 because the PHPUnit version is ~4 in composer.json |
Ok, I'll check it out. I will have to update the test suite. Will do
this weekend.
…On Fri, Jan 28, 2022, 12:58 PM Gareth Ellis ***@***.***> wrote:
It doesn't look like it's possible to run the test suite on PHP 8.1
because the PHPUnit version is ~4 in composer.json
—
Reply to this email directly, view it on GitHub
<#347 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADRTNSVHUCBSCMZP7IFGLDUYLKLBANCNFSM5LHZOS6Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
hi @greenlion I had a go at upgrading phpunit and making the test suite pass on php 8.1 without deprecations: |
Co-authored-by: Gareth Ellis <gareth.ellis@iconplc.com>
Thank you everybody for your help. All three fixes are merged today. |
Thanks! Do you have any plans for when you might tag this as a release? |
I can do that later today.
…On Tue, Feb 1, 2022, 4:17 AM Gareth Ellis ***@***.***> wrote:
Thanks! Do you have any plans for when you might tag this as a release?
—
Reply to this email directly, view it on GitHub
<#347 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADRTNUK466VT5Z5HR5R44TUY6QJJANCNFSM5LHZOS6Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
Done.
…On Tue, Feb 1, 2022 at 4:48 AM Justin Swanhart ***@***.***> wrote:
I can do that later today.
On Tue, Feb 1, 2022, 4:17 AM Gareth Ellis ***@***.***>
wrote:
> Thanks! Do you have any plans for when you might tag this as a release?
>
> —
> Reply to this email directly, view it on GitHub
> <#347 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AADRTNUK466VT5Z5HR5R44TUY6QJJANCNFSM5LHZOS6Q>
> .
> Triage notifications on the go with GitHub Mobile for iOS
> <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
> or Android
> <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
>
> You are receiving this because you modified the open/close state.Message
> ID: ***@***.***>
>
|
Hi guys!
Have some deprecations on PHP 8.1:
Deprecated: preg_split(): Passing null to parameter #3 ($limit) of type int is deprecated in /development/application/vendor/greenlion/php-sql-parser/src/PHPSQLParser/lexer/PHPSQLLexer.php on line 87
Deprecated: Automatic conversion of false to array is deprecated in /development/application/vendor/greenlion/php-sql-parser/src/PHPSQLParser/processors/SQLProcessor.php on line 506
and so on.
Could you fix these?
The text was updated successfully, but these errors were encountered: