Skip to content

Improve PHP 8.4+ support by avoiding implicitly nullable types - #87

Merged
WyriHaximus merged 1 commit into
reactphp:4.xfrom
clue-labs:nullable-v4
May 24, 2024
Merged

Improve PHP 8.4+ support by avoiding implicitly nullable types#87
WyriHaximus merged 1 commit into
reactphp:4.xfrom
clue-labs:nullable-v4

Conversation

@clue

@clueclue commented May 22, 2024

Copy link
Copy Markdown
Member

This changeset improves PHP 8.4+ support by avoiding implicitly nullable types as discussed in reactphp/promise#260.

Marking this as WIP until the upstream components have been released. In the meantime, this may use temporary development branches. This change only affects v4, so there is no need to backport these changes to v3 or v2.

Builds on top of #86, #81, #15 and reactphp/promise#260

@clueclue added the new feature New feature or request label May 22, 2024
@clueclue added this to the v4.3.0 milestone May 22, 2024
@clue
clue marked this pull request as ready for review May 24, 2024 14:05
@clueclue changed the title [WIP] Improve PHP 8.4+ support by avoiding implicitly nullable typesImprove PHP 8.4+ support by avoiding implicitly nullable typesMay 24, 2024
@clue

clue commented May 24, 2024

Copy link
Copy Markdown
MemberAuthor

Updated now that https://github.com/reactphp/promise/releases/tag/v3.2.0 has been released, this is now ready for review :shipit:

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

Nice, looks good to me 👍

@WyriHaximus
WyriHaximus merged commit 0c60cd1 into reactphp:4.xMay 24, 2024
@clue
clue deleted the nullable-v4 branch May 24, 2024 20:00
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new featureNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@clue@WyriHaximus@SimonFrings