Skip to content

Commit 6f49bfb

Browse files
authored
toxgen: Make it clearer which suites can be migrated (#4196)
...also, `cohere` was in the `IGNORE` list twice, apparently.
1 parent d394ef6 commit 6f49bfb

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

scripts/populate_tox/populate_tox.py

+8-4
Original file line numberDiff line numberDiff line change
@@ -49,22 +49,26 @@
4949
# suites over to this script. Some entries will probably stay forever
5050
# as they don't fit the mold (e.g. common, asgi, which don't have a 3rd party
5151
# pypi package to install in different versions).
52+
#
53+
# Test suites that will have to remain hardcoded since they don't fit the
54+
# toxgen usecase
55+
"asgi",
56+
"aws_lambda",
57+
"cloud_resource_context",
5258
"common",
5359
"gevent",
5460
"opentelemetry",
5561
"potel",
62+
# Integrations that can be migrated -- we should eventually remove all
63+
# of these from the IGNORE list
5664
"aiohttp",
5765
"anthropic",
5866
"arq",
59-
"asgi",
6067
"asyncpg",
61-
"aws_lambda",
6268
"beam",
6369
"boto3",
6470
"chalice",
6571
"cohere",
66-
"cloud_resource_context",
67-
"cohere",
6872
"django",
6973
"fastapi",
7074
"gcp",

0 commit comments

Comments
 (0)