Skip to content

Test on PHP 8.2 - #466

Merged
clue merged 1 commit into
reactphp:1.xfrom
WyriHaximus-labs:1.x-test-on-php-8.2
Aug 23, 2022
Merged

Test on PHP 8.2#466
clue merged 1 commit into
reactphp:1.xfrom
WyriHaximus-labs:1.x-test-on-php-8.2

Conversation

@WyriHaximus

@WyriHaximusWyriHaximus commented Aug 13, 2022

Copy link
Copy Markdown
Member

With PHP 8.2 coming out later this year, we should be reading for it's release to ensure all out code works on it.

Refs: reactphp/event-loop#258
Builds on top of #459

@WyriHaximusWyriHaximus added this to the v1.7.0 milestone Aug 13, 2022
@WyriHaximus
WyriHaximus marked this pull request as ready for review August 23, 2022 05:38
@WyriHaximus
WyriHaximus requested a review from clueAugust 23, 2022 05:38
@WyriHaximus

Copy link
Copy Markdown
MemberAuthor

@clue With a fix from PHPUnit's side yesterday, all checks are now green 🎉

With PHP 8.2 coming out later this year, we should be reading for it's release to ensure all out code works on it.
Refs: reactphp/event-loop#258

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

@WyriHaximus Thanks for the update, changes LGTM! :shipit:

@clue
clue merged commit 5f3b58c into reactphp:1.xAug 23, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@WyriHaximus@clue