Skip to content

fix(pubsub) : resolve issue with pubsub generation#17219

Merged
parthea merged 3 commits into
mainfrom
test/verify-pubsub-generation
May 22, 2026
Merged

fix(pubsub) : resolve issue with pubsub generation#17219
parthea merged 3 commits into
mainfrom
test/verify-pubsub-generation

Conversation

@chalmerlowe
Copy link
Copy Markdown
Contributor

@chalmerlowe chalmerlowe commented May 21, 2026

Corrects issue with package generation.

Changes

librarian.yaml:

  • Enabled client library generation for google-cloud-pubsub by removing skip_generate.

.librarian/generator-input/client-post-processing/pubsub-integration.yaml:

  • Removed obsolete grpcio-status post-processing rules that are no longer necessary.
  • Added a unified post-processing rule to correctly split an unintentionally merged/concatenated grpcio dependency constraint back into two separate, version-constrained lines for use in setup.py.

packages/google-cloud-pubsub/pytest.ini:

  • Added a selective warning ignore rule (ignore:.*SelectableGroups dict interface is deprecated:DeprecationWarning) to prevent pytest from failing on third-party dependency deprecation warnings under Python 3.11+.

packages/google-cloud-pubsub/README.rst & packages/google-cloud-pubsub/docs/README.rst:

  • README files updated due to changes in synthtool template now reflect the dropped support for Python 3.9 (minimum version bounds bumped from >= 3.9 to >= 3.10).

@chalmerlowe chalmerlowe requested review from a team as code owners May 21, 2026 13:43
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 supported Python versions for the google-cloud-pubsub library, dropping support for Python 3.9 and adding explicit compatibility for Python 3.14. It also corrects a formatting error in setup.py where dependency lines were merged and adds a warning filter to pytest.ini. Feedback identifies significant issues in the post-processing YAML configuration, specifically regarding a trailing newline bug that could break setup.py generation, the use of hardcoded versions instead of regex backreferences, and the removal of a relevant issue link.

Comment thread librarian.yaml Outdated
apis:
- path: google/pubsub/v1
skip_generate: true
skip_generate: false
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you remove this line. The default is false. Conciseness is good.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

Comment thread librarian.yaml Outdated
Comment thread packages/google-cloud-pubsub/README.rst
@parthea parthea marked this pull request as draft May 21, 2026 15:38
@chalmerlowe chalmerlowe marked this pull request as ready for review May 21, 2026 22:38
@chalmerlowe chalmerlowe added this to the generate via 1.32.0 milestone May 22, 2026
@parthea parthea merged commit b21c1d4 into main May 22, 2026
35 checks passed
@parthea parthea deleted the test/verify-pubsub-generation branch May 22, 2026 14:43
@chalmerlowe chalmerlowe changed the title fix: resolve issue with pubsub generation fix(pusub) : resolve issue with pubsub generation May 22, 2026
@chalmerlowe chalmerlowe changed the title fix(pusub) : resolve issue with pubsub generation fix(pubsub) : resolve issue with pubsub generation May 22, 2026
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.

4 participants