- Notifications
You must be signed in to change notification settings - Fork90
feat: add support for python 3.13#696
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
parthea commentedSep 13, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
|
@@ -95,7 +95,8 @@ def install_prerelease_dependencies(session, constraints_path): | |||
prerel_deps = [ | |||
"google-auth", | |||
"googleapis-common-protos", | |||
"grpcio", | |||
# Exclude grpcio!=1.67.0rc1 which does not support python 3.13 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Do we need to disable therc
version? My assumption is that the latest stable version is installed by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Without this change, the pre-release test fails (Seehttps://github.com/googleapis/python-api-core/actions/runs/11259852849/job/31309789274)
Do we need to disable the rc version? My assumption is that the latest stable version is installed by default.
We don't get the stable version in theprerelease_deps
nox session. We get the latest pre-release which ishttps://pypi.org/project/grpcio/1.67.0rc1/. We're not disabling allrc
versions, just1.67.0rc1
which doesn't support python 3.13
Uh oh!
There was an error while loading.Please reload this page.
Co-authored-by: ohmayr <omairnaveed@ymail.com>
46b3d3a
intomainUh oh!
There was an error while loading.Please reload this page.
Fixes#695 🦕