-
Notifications
You must be signed in to change notification settings - Fork 519
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
chore: add testruns for PHP 8.4 #1235
Conversation
Thanks for doing this. @jbelien What's needed to get this merged? Anything I can do to help? |
@Nyholm Would you be able to help out with this? |
There is one failing test for PHP 8.4 low. I guess we need to bump the minimum version of http-client? v7.0.10 ? |
Since We can change it to: -"symfony/http-client": "^5.4 || ^6.4 || ^7.0",
+"symfony/http-client": "^5.4 || ^6.4 || ^7.0.10", |
@Chris53897 could you make the change? |
I bumped the min version to ^5.4.45 to solve it. |
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.
I think it looks good
@Chris53897 Looks good, only a few PHP CS Fixer issues left and we can ship it! 🎉 |
I have created a seperate PR for this.Am 14.11.2024 um 10:04 schrieb Ruud Kamphuis ***@***.***>:
@Chris53897 Looks good, only a few PHP CS Fixer issues left and we can ship it! 🎉
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Why, would have been easier for everyone if it was done in this one, so we could have just merged it and be done with it. Now you have to rebase this PR later again. |
No description provided.