Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
9df84a5 to357ce21Compare9e7fa10 tob5894a6Compare
alexandre-daubois left a comment
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.
🎉
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Neirda24 commentedJun 3, 2025
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 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, 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 to2932863Compare6251fbc intosymfony:8.0Uh oh!
There was an error while loading.Please reload this page.
vinceAmstoutz commentedJun 5, 2025
🎉🔥 |
garak commentedJun 11, 2025
https://symfony.com/releases/8.0 still reports "8.2.0 or higher" as requirement tho |
kbond commentedJun 11, 2025
I think this is something@javiereguiluz can update. |
javiereguiluz commentedJun 12, 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
Uh oh!
There was an error while loading.Please reload this page.
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