Skip to content

refactor: Refactor endpoints #7

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

Closed
wants to merge 11 commits into from
Closed

refactor: Refactor endpoints #7

wants to merge 11 commits into from

Conversation

edenreich
Copy link
Collaborator

@edenreich edenreich commented Mar 20, 2025

Summary

This Pull-Request bringing the rust-sdk to be OpenAI Compatible.

BREAKING CHANGE: if you're running clients that using OpenAI sdks there are not much breaking changes in term of data structure, but if you used this library for a while, the changes are now moved in favor to support OpenAI data structure. I've went with this decision because many other tools are more standardized towards OpenAI, which making the transition seamlessly.

Signed-off-by: Eden Reich <eden.reich@gmail.com>
Signed-off-by: Eden Reich <eden.reich@gmail.com>
I will probably implement Google very soon, because I found out they have an OpenAI compatible API.

Signed-off-by: Eden Reich <eden.reich@gmail.com>
…emove ssevents field

Signed-off-by: Eden Reich <eden.reich@gmail.com>
Signed-off-by: Eden Reich <eden.reich@gmail.com>
Signed-off-by: Eden Reich <eden.reich@gmail.com>
…pletion

Signed-off-by: Eden Reich <eden.reich@gmail.com>
…completion

Signed-off-by: Eden Reich <eden.reich@gmail.com>
Signed-off-by: Eden Reich <eden.reich@gmail.com>
… argument parsing

Signed-off-by: Eden Reich <eden.reich@gmail.com>
Otherwise it's conflicting with the trait Default that could be implemented. This approach seems more readable to me so I'll go with this approach.

Signed-off-by: Eden Reich <eden.reich@gmail.com>
@edenreich edenreich closed this Mar 20, 2025
@edenreich edenreich deleted the refactor-endpoints branch March 20, 2025 18:46
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.

None yet

1 participant