Skip to content

Nate/fix-openai-adapters #5612

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Nate/fix-openai-adapters #5612

wants to merge 4 commits into from
+52 −6

Conversation

sestinj
Copy link
Contributor

@sestinj sestinj commented May 11, 2025 •
edited by mrge-io bot
Loading

Description

Fixes openai-adapters for deepseek and mercury, adds tests

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Summary by mrge

Fixed configuration issues in the DeepSeek and Inception OpenAI adapters and added tests to verify correct setup.

  • Bug Fixes

    • Ensured DeepSeek and Inception adapters use the correct default API base URLs and handle custom API base overrides.

  • Tests

    • Added tests to check adapter configuration for API base and API key.

@sestinj sestinj requested a review from a team as a code owner May 11, 2025 02:25
@sestinj sestinj requested review from tomasz-stefaniak and removed request for a team May 11, 2025 02:25
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label May 11, 2025
@netlify Netlify
Copy link

netlify bot commented May 11, 2025 •
edited
Loading

✅ Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 6d85b83
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/68200c7b77a2710008ac3b18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomasz-stefaniak tomasz-stefaniak Awaiting requested review from tomasz-stefaniak tomasz-stefaniak is a code owner automatically assigned from continuedev/continue-code-reviewers

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
size:M This PR changes 30-99 lines, ignoring generated files.
Projects
Issues and PRs
Status: Todo
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@sestinj