Skip to content

fix(google-auth): add extra for grpc#17644

Merged
parthea merged 3 commits into
mainfrom
add-grpc-extra-and-cleanup-todos
Jul 7, 2026
Merged

fix(google-auth): add extra for grpc#17644
parthea merged 3 commits into
mainfrom
add-grpc-extra-and-cleanup-todos

Conversation

@ohmayr

@ohmayr ohmayr commented Jul 7, 2026

Copy link
Copy Markdown
Contributor
  • Added grpc extra ("grpc": ["grpcio"]) and updated testing_extra_require to use *grpc_extra_require.
  • Removed obsolete TODO comments for grpcio, oauth2client, and urllib3 bounds.

@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 packages/google-auth/setup.py to introduce a new grpc extra requirement containing grpcio, which is now reused within the testing extra requirements. Additionally, it removes several obsolete TODO comments regarding dependency management. As there are no review comments, no further feedback is provided.

@ohmayr ohmayr marked this pull request as ready for review July 7, 2026 16:07
@ohmayr ohmayr requested review from a team as code owners July 7, 2026 16:07
@parthea parthea marked this pull request as draft July 7, 2026 16:35
@ohmayr ohmayr marked this pull request as ready for review July 7, 2026 16:36
@parthea parthea enabled auto-merge (squash) July 7, 2026 16:42
@parthea parthea changed the title chore(google-auth): add gRPC extra and clean up obsolete TODOs fix(google-auth): add extra for grpc Jul 7, 2026
Comment thread packages/google-auth/setup.py
@parthea parthea merged commit 6da41e8 into main Jul 7, 2026
30 checks passed
@parthea parthea deleted the add-grpc-extra-and-cleanup-todos branch July 7, 2026 17:04
@release-please release-please Bot mentioned this pull request Jul 7, 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.

2 participants