Skip to content
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.4] Fixes for implicit nullability deprecation #255

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

Ayesh
Copy link
Contributor

@Ayesh Ayesh commented Mar 15, 2024

Fixes all issues that emits a deprecation notice on PHP 8.4.

See:

Fixes all issues that emits a deprecation notice on PHP 8.4.

See:
 - [RFC](https://wiki.php.net/rfc/deprecate-implicitly-nullable-types)
 - [PHP 8.4: Implicitly nullable parameter declarations deprecated](https://php.watch/versions/8.4/implicitly-marking-parameter-type-nullable-deprecated)
@nicolas-grekas
Copy link
Collaborator

@Zegnat @Nyholm sorry for the ping: could you please consider merging this PR ASAP? That'd help on Symfony's side! 🙏

@Zegnat
Copy link
Collaborator

Zegnat commented Apr 12, 2024

No worries and thanks for the ping @nicolas-grekas! Answering from my phone now, day job ahead (CEST), but I can commit some time as soon as I get to my home machine if nobody beats me to it.

@nicolas-grekas
Copy link
Collaborator

Ah, I just realized I do have merge rights! 😅

@nicolas-grekas nicolas-grekas merged commit 229484f into Nyholm:master Apr 12, 2024
11 of 12 checks passed
@nicolas-grekas
Copy link
Collaborator

nicolas-grekas commented Apr 12, 2024

Merged (I'll let you do the releasing, it's not a hurry on Symfony's side, we run dev deps :))

@davidbyoung
Copy link

Any chance we could get a stable release that includes these changes? Thanks!

@Nyholm
Copy link
Owner

Nyholm commented Sep 9, 2024

Of course. Here you go

https://github.com/Nyholm/psr7/releases/tag/1.8.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants