Skip to content

[HttpClient] Fix low-deps job #60734

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

Merged
merged 1 commit into from
Jun 6, 2025
Merged

Conversation

nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Jun 6, 2025

Q A
Branch? 8.0
Bug fix? no
New feature? no
Deprecations? no
Issues -
License MIT

We need the fix for amphp/http-client#390

@ro0NL
Copy link
Contributor

ro0NL commented Jun 6, 2025

ref #60733 #60417

💯

@ro0NL
Copy link
Contributor

ro0NL commented Jun 6, 2025

does it apply to 6.4 as well?

@nicolas-grekas nicolas-grekas changed the title [HttpClient] Tentatively fix low-deps job [HttpClient] Fix low-deps job Jun 6, 2025
@nicolas-grekas
Copy link
Member Author

No need to fix anything on 6.4.

@ro0NL
Copy link
Contributor

ro0NL commented Jun 6, 2025

@nicolas-grekas nicolas-grekas merged commit cc19652 into symfony:8.0 Jun 6, 2025
7 of 8 checks passed
@nicolas-grekas
Copy link
Member Author

For another reason, not only on low-deps e.g.

@nicolas-grekas nicolas-grekas deleted the hc-low branch June 6, 2025 15:51
@ro0NL
Copy link
Contributor

ro0NL commented Jun 6, 2025

and this is why i opened #60733

@nicolas-grekas
Copy link
Member Author

nicolas-grekas commented Jun 6, 2025

This is not a chat...

You linked, https://github.com/symfony/symfony/actions/runs/15493301193/job/43623792379, which is already mentioned in #60417 so #60733 is a duplicate.

Please verify before expecting others (me here) to do it.

@ro0NL
Copy link
Contributor

ro0NL commented Jun 6, 2025

8.0 is still red https://github.com/symfony/symfony/tree/8.0, as are other branches

@ro0NL ro0NL mentioned this pull request Jun 6, 2025
@nicolas-grekas
Copy link
Member Author

Sure, unrelated to HttpClient though. We discussed that with @mtarld already as that looks related to the new type system.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants