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
Upgrade symfony http-kernel to 5.4.31 #34524
Upgrade symfony http-kernel to 5.4.31 #34524
Conversation
Hlavtox
commented
Nov 10, 2023
•
edited
edited
Questions | Answers |
---|---|
Branch? | develop |
Description? | My dear friend @ShaiMagal made sure that putenv no longer throws an error on public hostings in symfony/symfony#52428, this allows to benefit from it. |
Type? | refacto |
Category? | CO |
BC breaks? | no |
Deprecations? | no |
How to test? | |
UI Tests | https://github.com/Hlavtox/ga.tests.ui.pr/actions/runs/6827395441 |
Fixed issue or discussion? | |
Related PRs | |
Sponsor company |
👍 but there is a cron for that, right? |
@nicosomb I am not sure, why we would still be on 5.4.23 then? |
The release is just some hours ago so the cron has simply not run yet |
@matks Ok, so why we are not on 5.4.30 that was released 2 weeks ago? :D https://github.com/symfony/http-kernel/releases/tag/v5.4.30 |
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.
OK for me. As discussed with @Progi1984 and @matthieu-rolland, this kind of pull request should be handled by dependabot.
@Hlavtox could you restart the failing tests please? |
@nicosomb Done, only failing tests are with some wishlist, but I test it locally and everything works, no errors, no notices, so probably a false alarm. |
Thank you @Hlavtox and @nesrineabdmouleh ! |