Skip to content

tests: Move anthropic under toxgen #4348

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

Merged
merged 4 commits into from
Apr 29, 2025
Merged

tests: Move anthropic under toxgen #4348

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2cc39ea
tests: Update tox.ini
sentrivana Apr 29, 2025
0beed64
tests: Move anthropic under toxgen
sentrivana Apr 29, 2025
b73b738
adjust versions
sentrivana Apr 29, 2025
04f99f7
Merge branch 'master' into ivana/toxgen/move-more-stuff
sentrivana Apr 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/test-integrations-ai.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ jobs:
strategy:
fail-fast: false
matrix:
python-version: ["3.7","3.9","3.11","3.12"]
python-version: ["3.9","3.11","3.12"]
# python3.6 reached EOL and is no longer being supported on
# new versions of hosted runners on Github Actions
# ubuntu-20.04 is the last version that supported python3.6
Expand Down
8 changes: 8 additions & 0 deletions scripts/populate_tox/config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,14 @@
},
"python": ">=3.7",
},
"anthropic": {
"package": "anthropic",
"deps": {
"*": ["pytest-asyncio"],
"<0.50": ["httpx<0.28.0"],
},
"python": ">=3.8",
},
"ariadne": {
"package": "ariadne",
"deps": {
Expand Down
1 change: 0 additions & 1 deletion scripts/populate_tox/populate_tox.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@
"potel",
# Integrations that can be migrated -- we should eventually remove all
# of these from the IGNORE list
"anthropic",
"arq",
"asyncpg",
"beam",
Expand Down
12 changes: 0 additions & 12 deletions scripts/populate_tox/tox.jinja
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,10 +36,6 @@ envlist =
# At a minimum, we should test against at least the lowest
# and the latest supported version of a framework.

# Anthropic
{py3.8,py3.11,py3.12}-anthropic-v{0.16,0.28,0.40}
{py3.7,py3.11,py3.12}-anthropic-latest

# Arq
{py3.7,py3.11}-arq-v{0.23}
{py3.7,py3.12,py3.13}-arq-latest
Expand Down Expand Up @@ -179,14 +175,6 @@ deps =

# === Integrations ===

# Anthropic
anthropic: pytest-asyncio
anthropic-v{0.16,0.28}: httpx<0.28.0
anthropic-v0.16: anthropic~=0.16.0
anthropic-v0.28: anthropic~=0.28.0
anthropic-v0.40: anthropic~=0.40.0
anthropic-latest: anthropic

# Arq
arq-v0.23: arq~=0.23.0
arq-v0.23: pydantic<2
Expand Down
28 changes: 15 additions & 13 deletions tox.ini
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
# The file (and all resulting CI YAMLs) then need to be regenerated via
# "scripts/generate-test-files.sh".
#
# Last generated: 2025-04-29T08:15:04.584844+00:00
# Last generated: 2025-04-29T08:35:44.624881+00:00

[tox]
requires =
Expand All @@ -36,10 +36,6 @@ envlist =
# At a minimum, we should test against at least the lowest
# and the latest supported version of a framework.

# Anthropic
{py3.8,py3.11,py3.12}-anthropic-v{0.16,0.28,0.40}
{py3.7,py3.11,py3.12}-anthropic-latest
Copy link
Member

@szokeasaurusrex szokeasaurusrex Apr 29, 2025

Choose a reason for hiding this comment

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

It looks like we don't test any py3.7 versions anymore, is that intentional?

Copy link
Contributor Author

@sentrivana sentrivana Apr 29, 2025

Choose a reason for hiding this comment

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

Yeah, this is an unfortunate consequence of some version conflicts newer anthropic packages have when you try to install them on 3.7: https://github.com/getsentry/sentry-python/actions/runs/14726662709/job/41330837840
Only the oldest release is installable on 3.7. I think it's fine to start testing on 3.8+.


# Arq
{py3.7,py3.11}-arq-v{0.23}
{py3.7,py3.12,py3.13}-arq-latest
Expand Down Expand Up @@ -139,6 +135,11 @@ envlist =
# integration tests there.

# ~~~ AI ~~~
{py3.8,py3.11,py3.12}-anthropic-v0.16.0
{py3.8,py3.11,py3.12}-anthropic-v0.27.0
{py3.8,py3.11,py3.12}-anthropic-v0.38.0
{py3.8,py3.11,py3.12}-anthropic-v0.50.0

{py3.9,py3.10,py3.11}-cohere-v5.4.0
{py3.9,py3.11,py3.12}-cohere-v5.8.1
{py3.9,py3.11,py3.12}-cohere-v5.11.4
Expand Down Expand Up @@ -335,14 +336,6 @@ deps =

# === Integrations ===

# Anthropic
anthropic: pytest-asyncio
anthropic-v{0.16,0.28}: httpx<0.28.0
anthropic-v0.16: anthropic~=0.16.0
anthropic-v0.28: anthropic~=0.28.0
anthropic-v0.40: anthropic~=0.40.0
anthropic-latest: anthropic

# Arq
arq-v0.23: arq~=0.23.0
arq-v0.23: pydantic<2
Expand Down Expand Up @@ -506,6 +499,15 @@ deps =
# integration tests there.

# ~~~ AI ~~~
anthropic-v0.16.0: anthropic==0.16.0
anthropic-v0.27.0: anthropic==0.27.0
anthropic-v0.38.0: anthropic==0.38.0
anthropic-v0.50.0: anthropic==0.50.0
anthropic: pytest-asyncio
anthropic-v0.16.0: httpx<0.28.0
anthropic-v0.27.0: httpx<0.28.0
anthropic-v0.38.0: httpx<0.28.0

cohere-v5.4.0: cohere==5.4.0
cohere-v5.8.1: cohere==5.8.1
cohere-v5.11.4: cohere==5.11.4
Expand Down
Loading