-
Notifications
You must be signed in to change notification settings - Fork 14
feat(provider): add GitHub Models provider (chat + embeddings) #202
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
base: main
Are you sure you want to change the base?
feat(provider): add GitHub Models provider (chat + embeddings) #202
Conversation
|
Just a small note for the future, if you could use GitHub Keywords when opening PRs to connect it to the proper issue (to automate the entire process a bit more), it would be super helpful to us. You can read a bit more about it here (I've added closing keyword for this particular issue). |
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.
Hey @pratyushuniv2022-code, thanks for the contribution! 🚀
Did you gave a look at already existing providers on how they have been implemented?
for ref
Hey @Malaydewangan09 and @MilosPaunovic just to confirm the expected implementation style before I proceed: |
|
Yes, @pratyushuniv2022-code that would be great, thanks! |
|
Hello @Pratyushstark 👋 Any update on this one? |
|
Hey @pratyushuniv2022-code 👋, are there any updates on this? |
Closes #171.
Summary:
Notes:
./gradlew build -x test. Integration tests that require Docker were not run locally.PR readiness checklist
./gradlew clean build -x test✅./gradlew test --tests "io.kestra.plugin.ai.provider.github.GitHubProviderTest"✅examples/github-provider-example.yaml✅{{ secrets.GITHUB_MODELS_TOKEN }}✅