Skip to content

Conversation

@cpsievert
Copy link
Collaborator

No description provided.

@cpsievert cpsievert requested a review from Copilot August 12, 2025 20:20
@cpsievert cpsievert marked this pull request as ready for review August 12, 2025 20:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for the DeepSeek AI platform by implementing a new ChatDeepSeek provider class that integrates with the existing chatlas framework.

  • Implements ChatDeepSeek class as a wrapper around the OpenAI provider with DeepSeek-specific configurations
  • Adds comprehensive test coverage for the new provider including streaming, tools, and turns interface
  • Updates documentation to include DeepSeek in the supported models list

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
chatlas/_provider_deepseek.py Implements the main ChatDeepSeek provider class with DeepSeek API integration
tests/test_provider_deepseek.py Adds comprehensive test suite for DeepSeek provider functionality
chatlas/init.py Exports the new ChatDeepSeek class in the main module
docs/get-started/models.qmd Adds DeepSeek to the supported models documentation table
docs/_quarto.yml Includes ChatDeepSeek in the documentation generation config
CHANGELOG.md Documents the new DeepSeek feature addition

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cpsievert cpsievert merged commit 6790988 into main Aug 12, 2025
7 checks passed
@cpsievert cpsievert deleted the feat/chat-deepseek branch August 12, 2025 20:24
@cpsievert cpsievert mentioned this pull request Aug 12, 2025
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