Skip to content

Add models_deepseek() function for listing available DeepSeek models#918

Open
jcrodriguez1989 wants to merge 2 commits intotidyverse:mainfrom
jcrodriguez1989:feature/models-deepseek
Open

Add models_deepseek() function for listing available DeepSeek models#918
jcrodriguez1989 wants to merge 2 commits intotidyverse:mainfrom
jcrodriguez1989:feature/models-deepseek

Conversation

@jcrodriguez1989
Copy link
Copy Markdown

Summary

Adds the models_deepseek() function to complete the DeepSeek provider implementation, bringing it in line with other providers that have models_*() functions.

Changes

  • New function: models_deepseek() that retrieves available models from the DeepSeek API
    • Accepts base_url, api_key, and credentials parameters (consistent with other providers)
  • Tests: Added test case "can list models" using the common test_models() helper

Consistency

This implementation follows the established pattern used by other providers:

  • Similar signature to models_openai(), models_google_gemini(), etc.
  • Integrated into the existing documentation structure

Testing

  • ✅ New test added: test_that("can list models", { test_models(models_deepseek) })

Copy link
Copy Markdown
Member

@hadley hadley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Can you please add a bullet to the top of NEWS.md? It should briefly describe the change and end with (@yourname, #issuenumber).

@jcrodriguez1989
Copy link
Copy Markdown
Author

Done.
Let me know if further changes are needed.
Related issue: #919

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.

2 participants