Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Bump Symfony 8 to PHP >= 8.4#60639

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
nicolas-grekas merged 1 commit intosymfony:8.0fromnicolas-grekas:php84
Jun 3, 2025

Conversation

@nicolas-grekas
Copy link
Member

@nicolas-grekasnicolas-grekas commentedJun 2, 2025
edited
Loading

QA
Branch?8.0
Bug fix?no
New feature?yes
Deprecations?no
Issues-
LicenseMIT

Let's start this new major cycle with fresh features shipped with PHP 8.4: native lazy objects, HTML5 dom parser, etc.

For reference, this took inspiration from a similar PR for 7.0:#50404

skmedix, mtarld, lyrixx, welcoMattic, nikophil, mbabker, OskarStark, dunglas, kaznovac, agustingomes, and 11 more reacted with thumbs up emojiloevgaard, welcoMattic, fbourigault, chou666, althaus, connorhu, dmaicher, yoeunes, ker0x, and xeurun reacted with heart emojimahono, Kocal, alexandre-daubois, kaznovac, agustingomes, Jean-Beru, welcoMattic, wkania, dmaicher, smnandre, and 2 more reacted with rocket emoji
Copy link
Member

@alexandre-dauboisalexandre-daubois left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

🎉

@Neirda24
Copy link
Contributor

Awesome ! Would it be worth waiting for 8.5 for this kind of feature ?https://wiki.php.net/rfc/closures_in_const_expr It could be wild for some features. Otherwise it would have to wait 2 years for symfony 9 before including such functionnalities in core. I know that the timing is short because 8.5 is planned for November I think...

@stof
Copy link
Member

stof commentedJun 3, 2025

@Neirda24 We alreadysupport using static closures in some attributes in Symfony 7.3. We don't need to make PHP 8.5 the minimum supported version to benefit from those, as long as the usage of a closure is not required when using the attribute. For instance,IsGranted added support for using a static closure, but kept support for using an Expression (same for theWhen constraint).

And PHP 8.5.0 will be will be approximately at the same time than Symfony 8.0.0. This does not qualify it as a candidate for the min version, as it would make it too hard to upgrade to it (many companies don't upgrade their prod to the.0 release of PHP, to give it more time to stabilize with a few patch versions before they upgrade prod environments)

Neirda24, xabbuh, alexandre-daubois, Kocal, COil, and OskarStark reacted with thumbs up emoji

@nicolas-grekasnicolas-grekasforce-pushed thephp84 branch 2 times, most recently from068e8eb to2932863CompareJune 3, 2025 12:08
@nicolas-grekasnicolas-grekas merged commit6251fbc intosymfony:8.0Jun 3, 2025
7 of 10 checks passed
@nicolas-grekasnicolas-grekas deleted the php84 branchJune 3, 2025 12:36
@vinceAmstoutz
Copy link
Contributor

🎉🔥

@garak
Copy link
Contributor

https://symfony.com/releases/8.0 still reports "8.2.0 or higher" as requirement tho

@kbond
Copy link
Member

https://symfony.com/releases/8.0 still reports "8.2.0 or higher" as requirement tho

I think this is something@javiereguiluz can update.

@javiereguiluz
Copy link
Member

@garak@kbond this is now fixed. Thanks for reporting this error.

xabbuh reacted with thumbs up emojigarak reacted with heart emoji

@fabpotfabpot mentioned this pull requestOct 27, 2025
GromNaN added a commit that referenced this pull requestNov 1, 2025
This PR was merged into the 8.0 branch.Discussion----------[Mailer][MicrosoftGraph] Bump to PHP >= 8.4| Q             | A| ------------- | ---| Branch?       | 8.0| Bug fix?      | no| New feature?  | no| Deprecations? | no| Issues        | -| License       | MITFor reference#60639Commits-------f1bab88 [Mailer][MicrosoftGraph] Bump to PHP >= 8.4
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@xabbuhxabbuhxabbuh left review comments

@OskarStarkOskarStarkOskarStark approved these changes

@alexandre-dauboisalexandre-dauboisalexandre-daubois approved these changes

@lyrixxlyrixxAwaiting requested review from lyrixxlyrixx is a code owner

@dunglasdunglasAwaiting requested review from dunglasdunglas is a code owner

@ycerutoycerutoAwaiting requested review from ycerutoyceruto is a code owner

@welcoMatticwelcoMatticAwaiting requested review from welcoMatticwelcoMattic is a code owner

@kbondkbondAwaiting requested review from kbondkbond is a code owner

@chalasrchalasrAwaiting requested review from chalasrchalasr is a code owner

@jderussejderusseAwaiting requested review from jderussejderusse is a code owner

Assignees

No one assigned

Projects

None yet

Milestone

8.0

Development

Successfully merging this pull request may close these issues.

11 participants

@nicolas-grekas@Neirda24@stof@vinceAmstoutz@garak@kbond@javiereguiluz@OskarStark@xabbuh@alexandre-daubois@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp