-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump Symfony 8 to PHP >= 8.4 #60639
Conversation
9df84a5
to
357ce21
Compare
9e7fa10
to
b5894a6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
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... |
@Neirda24 We already support 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, 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 |
068e8eb
to
2932863
Compare
🎉🔥 |
https://symfony.com/releases/8.0 still reports "8.2.0 or higher" as requirement tho |
I think this is something @javiereguiluz can update. |
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