Skip to content

fix: require Protobuf 6.33.5+ - #17743

Draft
parthea wants to merge 1 commit into
mainfrom
require-protobuf-6-33-5
Draft

fix: require Protobuf 6.33.5+#17743
parthea wants to merge 1 commit into
mainfrom
require-protobuf-6-33-5

Conversation

@parthea

@parthea parthea commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Towards b/420641246

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

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.

Code Review

This pull request updates the minimum protobuf version requirement to 6.33.5 across multiple packages and configuration files. It also removes support for the 'cpp' protobuf implementation and cleans up associated test configurations and legacy logic. A review comment suggests removing a redundant parameterized test case in packages/google-api-core/noxfile.py to improve code cleanliness.

Comment thread packages/google-api-core/noxfile.py Outdated
@parthea parthea self-assigned this Jul 17, 2026
@parthea
parthea force-pushed the require-protobuf-6-33-5 branch from e13937b to a007839 Compare July 17, 2026 18:00
@parthea
parthea force-pushed the require-protobuf-6-33-5 branch 6 times, most recently from 2cd8f53 to 20cc3d3 Compare July 28, 2026 00:28
@parthea
parthea force-pushed the require-protobuf-6-33-5 branch from 20cc3d3 to f81395e Compare July 28, 2026 00:30
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