Skip to content

chore: librarian release pull request: 20260409T112159Z#16596

Closed
chalmerlowe wants to merge 3 commits intomainfrom
librarian-20260409T112159Z
Closed

chore: librarian release pull request: 20260409T112159Z#16596
chalmerlowe wants to merge 3 commits intomainfrom
librarian-20260409T112159Z

Conversation

@chalmerlowe
Copy link
Copy Markdown
Contributor

PR created by the Librarian CLI to initialize a release. Merging this PR will auto trigger a release.

Librarian Version: v0.8.0
Language Image: us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator@sha256:234b9d1f2ddb057ed7ac6a38db0bf8163d839c65c6cf88ade52530cddebce59e

google-cloud-storage: 3.11.0

3.11.0 (2026-04-09)

Features

  • drop support for Python 3.8 (b762ffdc)

Bug Fixes

  • require protobuf >= 4.25.8, < 8.0.0 (b762ffdc)

fix(deps): require protobuf >= 4.25.8, < 8.0.0

feat: migrate core library code from split repo

Note: Python 3.8 is no longer supported.
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 updates the google-cloud-storage library to version 3.11.0, which includes raising the minimum supported Python version to 3.9 and updating protobuf dependencies. It also standardizes service_config entries across various API definitions in the .librarian/state.yaml file and removes a copyright header from it. Feedback indicates that the CHANGELOG.md for google-cloud-storage has an imprecise description for the Python version change, a malformed commit hash, and an inconsistent repository name in a version comparison URL that could result in a broken link.

Comment on lines +10 to +17
### Features

* drop support for Python 3.8 ([b762ffdcb7c9ceced8b67f7b225c52f38f96444f](https://github.com/googleapis/google-cloud-python/commit/b762ffdcb7c9ceced8b67f7b225c52f38f96444f))


### Bug Fixes

* require protobuf >= 4.25.8, < 8.0.0 ([b762ffdcb7c9ceced8b67f7b225c52f38f96444f](https://github.com/googleapis/google-cloud-python/commit/b762ffdcb7c9ceced8b67f7b225c52f38f96444f))
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.

high

There are a couple of issues with this changelog section:

  • Imprecise description: The entry 'drop support for Python 3.8' is slightly misleading. The README.rst file update indicates that the minimum supported version is now Python 3.9, which means support for Python 3.7 was also dropped. For clarity, consider updating this to 'raise minimum Python version to 3.9' or 'drop support for Python 3.7 and 3.8'.
  • Malformed commit hash: The commit hash b762ffdcb7c9ceced8b67f7b225c52f38f96444f appears to be invalid. A standard Git SHA-1 hash is 40 hexadecimal characters, but this one is 48. This will result in broken links. Please correct the hash. The PR description references b762ffdc, which might be related.


[1]: https://pypi.org/project/google-cloud-storage/#history

## [3.11.0](https://github.com/googleapis/google-cloud-python/compare/google-cloud-storage-v3.10.1...google-cloud-storage-v3.11.0) (2026-04-09)
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 repository in the version comparison URL has been changed to google-cloud-python. Previous release entries in this file point to python-storage. This inconsistency may lead to a broken link if this change is not intentional as part of a repository migration.

@chalmerlowe
Copy link
Copy Markdown
Contributor Author

This PR is one step in clearing Issue #16487, namely ensuring the changelog reflects dependency and runtime changes.

Other PRs that help to clear Issue #16487 include:

@parthea parthea added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 9, 2026
@parthea
Copy link
Copy Markdown
Contributor

parthea commented Apr 9, 2026

This is not ready since #16487 is still open

@parthea parthea closed this Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing. release:pending

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants