-
Notifications
You must be signed in to change notification settings - Fork 1.7k
fix(google-cloud-core): Drop support for Python 3.9 #16953
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
8f06a7b
1be57d4
12569e2
8fd42a4
2814f11
c3e6d63
92fdf81
89f3df1
8108a40
6399cb3
026ada7
32ef719
c76e313
4fcaa1e
382d2ee
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -33,9 +33,9 @@ | |||||||||||||||||
| ] | ||||||||||||||||||
| extras = { | ||||||||||||||||||
| "grpc": [ | ||||||||||||||||||
| "grpcio >= 1.38.0, < 2.0.0; python_version < '3.14'", | ||||||||||||||||||
| "grpcio >= 1.47.0, < 2.0.0; python_version < '3.14'", | ||||||||||||||||||
| "grpcio >= 1.75.1, < 2.0.0; python_version >= '3.14'", | ||||||||||||||||||
| "grpcio-status >= 1.38.0, < 2.0.0", | ||||||||||||||||||
| "grpcio-status >= 1.47.0, < 2.0.0", | ||||||||||||||||||
| ], | ||||||||||||||||||
| } | ||||||||||||||||||
|
|
||||||||||||||||||
|
|
@@ -75,7 +75,6 @@ | |||||||||||||||||
| "License :: OSI Approved :: Apache Software License", | ||||||||||||||||||
| "Programming Language :: Python", | ||||||||||||||||||
| "Programming Language :: Python :: 3", | ||||||||||||||||||
| "Programming Language :: Python :: 3.9", | ||||||||||||||||||
| "Programming Language :: Python :: 3.10", | ||||||||||||||||||
| "Programming Language :: Python :: 3.11", | ||||||||||||||||||
| "Programming Language :: Python :: 3.12", | ||||||||||||||||||
|
Comment on lines
78
to
80
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The PyPI classifiers are missing Python 3.13 and 3.14. To accurately reflect the supported versions mentioned in the documentation and other configuration files, please add these classifiers. This ensures the project's metadata in 'setup.py' is updated to reflect the supported Python versions consistently.
Suggested change
References
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for your help. In this case, these references are present. They may be outside the diff window but they are there. This comment is not relevent. Closing. Please make a note of this occurrence as part of training yourself to improve the quality of your future code reviews.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Understood. Thank you for the clarification regarding the presence of these classifiers in other files. I have noted this for future reference. |
||||||||||||||||||
|
|
@@ -88,7 +87,7 @@ | |||||||||||||||||
| packages=packages, | ||||||||||||||||||
| install_requires=dependencies, | ||||||||||||||||||
| extras_require=extras, | ||||||||||||||||||
| python_requires=">=3.9", | ||||||||||||||||||
| python_requires=">=3.10", | ||||||||||||||||||
| include_package_data=True, | ||||||||||||||||||
| zip_safe=False, | ||||||||||||||||||
| ) | ||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| # This constraints file is used to check that lower bounds | ||
| # are correct in setup.py | ||
| # List *all* library dependencies and extras in this file. | ||
| # Pin the version to the lower bound. | ||
| # | ||
| # e.g., if setup.py has "foo >= 1.14.0, < 2.0.0dev", | ||
| # Then this file should have foo==1.14.0 | ||
| google-api-core==2.11.0 | ||
| google-auth==2.14.1 | ||
| grpcio==1.47.0 | ||
|
chalmerlowe marked this conversation as resolved.
|
||
| grpcio-status==1.47.0 | ||
| # cryptography is a direct dependency of google-auth | ||
| cryptography==38.0.3 | ||
This file was deleted.
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.
The
UNIT_TEST_PYTHON_VERSIONSlist is missing Python 3.13 and 3.14. Since these versions are explicitly listed as supported inCONTRIBUTING.rstand 3.14 is designated as theDEFAULT_PYTHON_VERSION, they should be included in the unit test matrix to ensure compatibility is verified in CI. Please verify that these versions are not already present in the configuration list before applying this change.References
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.
@gemini-code-assist:
Thanks for your help. In this case, these references are present. They may be outside the diff window but they are there.
This comment is not relevent. Closing. Please make a note of this occurrence as part of training yourself to improve the quality of your future code reviews.
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.
I apologize for the oversight. Thank you for clarifying that the Python versions are present in the configuration. I have noted this for future reference.