Skip to content

Conversation

@njbrake
Copy link
Contributor

@njbrake njbrake commented Sep 10, 2025

Description

A whole bunch of stuff was stuffed into the providers.py file. I'm prepping work for the #381 Public API PR, and to make that PR cleaner I figured I would do this first re-org here.

The file structure will change a bit more when I implement the Public API PR, but this handles the initial cleanup

PR Type

💅 Refactor ## Relevant issues

Checklist

  • I have added unit tests that prove my fix/feature works
  • New and existing tests pass locally
  • Documentation was updated where necessary
  • I have read and followed the contribution guidelines```

@njbrake njbrake requested a review from daavoo September 10, 2025 08:55
@codecov
Copy link

codecov bot commented Sep 10, 2025

Codecov Report

❌ Patch coverage is 93.15068% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/any_llm/factory.py 85.71% 10 Missing ⚠️
Files with missing lines Coverage Δ
src/any_llm/__init__.py 100.00% <100.00%> (ø)
src/any_llm/api.py 32.74% <100.00%> (+11.12%) ⬆️
src/any_llm/config.py 100.00% <100.00%> (ø)
src/any_llm/constants.py 100.00% <100.00%> (ø)
src/any_llm/provider.py 73.91% <100.00%> (+12.87%) ⬆️
src/any_llm/providers/bedrock/bedrock.py 61.11% <100.00%> (ø)
src/any_llm/providers/gemini/base.py 70.94% <100.00%> (ø)
src/any_llm/providers/gemini/gemini.py 100.00% <100.00%> (ø)
src/any_llm/providers/llamafile/llamafile.py 100.00% <100.00%> (ø)
src/any_llm/providers/lmstudio/lmstudio.py 100.00% <100.00%> (ø)
... and 6 more

... and 79 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@njbrake njbrake merged commit 077fbd0 into main Sep 10, 2025
10 checks passed
@njbrake njbrake deleted the brake/reorg branch September 10, 2025 09:01
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