Skip to content

Comments

feat: several providers don't yet support python 3.14#781

Merged
njbrake merged 4 commits intomainfrom
py314
Jan 28, 2026
Merged

feat: several providers don't yet support python 3.14#781
njbrake merged 4 commits intomainfrom
py314

Conversation

@njbrake
Copy link
Contributor

@njbrake njbrake commented Jan 28, 2026

Description

PR Type

  • 🐛 Bug Fix

Relevant issues

Checklist

  • I understand the code I am submitting.
  • I have added unit tests that prove my fix/feature works
  • I have run this code locally and verified it fixes the issue.
  • New and existing tests pass locally
  • Documentation was updated where necessary
  • I have read and followed the contribution guidelines
  • AI Usage:
    • No AI was used.
    • AI was used for drafting/refactoring.
    • This is fully AI-generated.

AI Usage Information

  • AI Model used: Opus 4.5
  • AI Developer Tool used: AoE
  • Any other info you'd like to share:

I used AoE to make the git worktree so that I could easily branch and write the code.

When answering questions by the reviewer, please respond yourself, do not copy/paste the reviewer comments into an AI system and paste back its answer. We want to discuss with you, not your AI :)

@njbrake njbrake requested a review from daavoo January 28, 2026 15:18
@codecov
Copy link

codecov bot commented Jan 28, 2026

Codecov Report

❌ Patch coverage is 82.60870% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/any_llm/providers/voyage/voyage.py 80.00% 1 Missing and 1 partial ⚠️
src/any_llm/providers/watsonx/watsonx.py 83.33% 1 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
src/any_llm/any_llm.py 70.19% <100.00%> (-1.99%) ⬇️
src/any_llm/providers/voyage/voyage.py 83.87% <80.00%> (-2.34%) ⬇️
src/any_llm/providers/watsonx/watsonx.py 75.58% <83.33%> (-1.25%) ⬇️

... and 44 files with indirect coverage changes

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

Copy link
Contributor

@daavoo daavoo left a comment

Choose a reason for hiding this comment

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

Not a blocking strong opinion.

Do you know which providers don't support it?

It seems openai https://github.com/openai/openai-python/blob/main/pyproject.toml does

If our base dependencies support it, maybe it is better to do a runtime exception checking the python version for the providers that don't 🤔

@njbrake
Copy link
Contributor Author

njbrake commented Jan 28, 2026

@daavoo Ooo thank you that's a good point. It's a problem for voyageai and watson providers. I was defaulting to 'eh lets not support it until everybody does' but yeah lets make it more selective. BRB with a solution haha

- Updated `requires-python` in `pyproject.toml` to allow Python 3.11 and above.
- Added conditional imports for `voyageai` and `ibm-watsonx-ai` packages in their respective provider files to handle compatibility with Python 3.14 and above, including specific error messages for users.
- Enhanced error message in `AnyLLM` class to provide more context when required packages are missing.
@njbrake njbrake changed the title feat: we don't yet support python 3.14 feat: several providers don't yet support python 3.14 Jan 28, 2026
@njbrake njbrake merged commit 6176847 into main Jan 28, 2026
11 of 12 checks passed
@njbrake njbrake deleted the py314 branch January 28, 2026 16:55
@njbrake njbrake linked an issue Jan 28, 2026 that may be closed by this pull request
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.

Limit to < python 3.14

2 participants