Skip to content
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

Use symfony/http-client #1

Merged
merged 1 commit into from
Mar 18, 2023
Merged

Conversation

GromNaN
Copy link
Collaborator

@GromNaN GromNaN commented Mar 3, 2023

For a better integration with Symfony and the profiler panel, symfony/http-client should be used.

That is now possible thanks to openai-php/client#75

@nunomaduro
Copy link
Contributor

Currently working in a different project, so I won't have time to review openai-php/client#65 for the next couple days. Maybe we can adjust this pull request, to ship with Guzzle for now?

@GromNaN
Copy link
Collaborator Author

GromNaN commented Mar 7, 2023

At the current state of the main branch, the bundle is working using guzzle. It can be tagged. This PR is an improvement for a better integration with Symfony toolchain.

@nunomaduro

This comment was marked as resolved.

@gehrisandro
Copy link
Collaborator

openai-php/client v0.4.0 has been released which removes the guzzle dependency and allows for a custom HTTP client: openai-php/client@v0.3.5...v0.4.0#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R51

@nunomaduro
Copy link
Contributor

cc @GromNaN

@GromNaN
Copy link
Collaborator Author

GromNaN commented Mar 18, 2023

Thank you for the update. This PR is ready.

@nunomaduro nunomaduro merged commit 916e195 into openai-php:main Mar 18, 2023
@GromNaN GromNaN deleted the symfony-http-client branch March 19, 2023 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants