Skip to content

[testing] for-testing-purposes-asset#17142

Draft
parthea wants to merge 1 commit into
mainfrom
testing-asset
Draft

[testing] for-testing-purposes-asset#17142
parthea wants to merge 1 commit into
mainfrom
testing-asset

Conversation

@parthea
Copy link
Copy Markdown
Contributor

@parthea parthea commented May 14, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

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.

Comment on lines +22 to +26
"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",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

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
  1. 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.

Comment on lines +40 to +41
google-api-core==2.17.1
google-auth==2.14.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

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
  1. 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.

Comment on lines 34 to 36
"3.10",
"3.11",
"3.12",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Since the project is adding dependency markers for Python 3.13 and 3.14 in setup.py, it is recommended to add Python 3.13 to the ALL_PYTHON list to ensure it is included in the test matrix.

Suggested change
"3.10",
"3.11",
"3.12",
"3.10",
"3.11",
"3.12",
"3.13",

Comment on lines 391 to 394
"sphinx-build",
"-W", # warnings as errors
"-T", # show full traceback on exception
"-N", # no colors
"-b",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

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.

Suggested change
"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",

Comment on lines 86 to 88
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

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.

Suggested change
"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",

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant