-
Notifications
You must be signed in to change notification settings - Fork 201
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
feat: Support openai base_url #180
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #180 +/- ##
==========================================
+ Coverage 77.95% 77.96% +0.01%
==========================================
Files 42 42
Lines 2118 2119 +1
==========================================
+ Hits 1651 1652 +1
Misses 467 467 ☔ View full report in Codecov by Sentry. |
@bdqfork thanks for the PR — could you fix the linting issues? We have instructions in contributor guidelines (number 5), ie:
Thanks! |
@jamescalam Done! |
@jamescalam Are there any other questions? |
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.
lgtm, thanks @bdqfork
Support setting the OpenAI client base URL to enable OpenAI-style LLM (Language Model) services.