[testing] for-testing-purposes-asset#17142
Conversation
There was a problem hiding this comment.
Code Review
This pull request drops support for Python 3.9, updates dependency lower bounds, and fixes missing await keywords in async samples and docstrings. It also updates the Librarian configuration to enable code generation for specific libraries and simplifies unit tests by removing legacy compatibility code. Feedback focuses on improving the robustness of Librarian post-processing rules using regex, ensuring Python 3.13 and 3.14 are fully represented in the test matrix and package metadata, and maintaining documentation quality by restoring the strict warnings-as-errors flag.
| "google-api-core\[grpc\] >= 2.17.1, <3.0.0", | ||
| # Exclude incompatible versions of `google-auth` | ||
| # See https://github.com/googleapis/google-cloud-python/issues/12364 | ||
| "google-auth >= 2.14.1, <3.0.0,!=2.24.0,!=2.25.0", | ||
| "grpcio >= 1.44.0, < 2.0.0", |
There was a problem hiding this comment.
The replacement pattern uses hardcoded version strings (e.g., 2.17.1, 2.14.1, 1.44.0). Per the general rules, flexible regex patterns should be used instead to ensure the replacement logic remains functional when dependency versions are updated in the source files.
"google-api-core\[grpc\] >= [\d.]+, <3.0.0",
# Exclude incompatible versions of `google-auth`
# See https://github.com/googleapis/google-cloud-python/issues/12364
"google-auth >= [\d.]+, <3.0.0,!=2.24.0,!=2.25.0",
"grpcio >= [\d.]+, < 2.0.0",References
- When defining search patterns for automated code replacements in librarian post-processing YAML files, use flexible regex patterns (such as [\s\S]*) to match version strings instead of hardcoding specific versions.
| google-api-core==2.17.1 | ||
| google-auth==2.14.1 |
There was a problem hiding this comment.
The replacement pattern uses hardcoded version strings. Per the general rules, flexible regex patterns should be used instead to maintain compatibility with future version updates.
google-api-core==[\d.]+
google-auth==[\d.]+References
- When defining search patterns for automated code replacements in librarian post-processing YAML files, use flexible regex patterns (such as [\s\S]*) to match version strings instead of hardcoding specific versions.
| "3.10", | ||
| "3.11", | ||
| "3.12", |
There was a problem hiding this comment.
| "sphinx-build", | ||
| "-W", # warnings as errors | ||
| "-T", # show full traceback on exception | ||
| "-N", # no colors | ||
| "-b", |
There was a problem hiding this comment.
The -W flag (warnings as errors) has been removed from the sphinx-build command. This allows documentation builds to succeed even if there are warnings (e.g., broken links, malformed docstrings), which can lead to a degradation in documentation quality over time. It is recommended to keep this flag enabled and address any specific warnings if they occur.
| "sphinx-build", | |
| "-W", # warnings as errors | |
| "-T", # show full traceback on exception | |
| "-N", # no colors | |
| "-b", | |
| "sphinx-build", | |
| "-W", # warnings as errors | |
| "-T", # show full traceback on exception | |
| "-N", # no colors | |
| "-b", |
| "Programming Language :: Python :: 3.10", | ||
| "Programming Language :: Python :: 3.11", | ||
| "Programming Language :: Python :: 3.12", |
There was a problem hiding this comment.
The project is adding dependency markers for Python 3.13 and 3.14 (lines 48, 50), but these versions are not listed in the classifiers. To fully reflect the supported versions, they should be included in the metadata.
| "Programming Language :: Python :: 3.10", | |
| "Programming Language :: Python :: 3.11", | |
| "Programming Language :: Python :: 3.12", | |
| "Programming Language :: Python :: 3.10", | |
| "Programming Language :: Python :: 3.11", | |
| "Programming Language :: Python :: 3.12", | |
| "Programming Language :: Python :: 3.13", | |
| "Programming Language :: Python :: 3.14", |
No description provided.