Skip to content

Conversation

parthea
Copy link
Collaborator

@parthea parthea commented Apr 8, 2024

Towards b/330968465

@product-auto-label product-auto-label bot added the size: xs Pull request size is extra small. label Apr 8, 2024
@parthea parthea force-pushed the add-api-version-header branch from c4498a7 to cbe0bb1 Compare April 8, 2024 16:05
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: xs Pull request size is extra small. labels Apr 8, 2024
@parthea parthea force-pushed the add-api-version-header branch 4 times, most recently from 578b720 to ad08619 Compare April 8, 2024 20:50
@parthea parthea changed the title feat: add google.api_core.gapic_v1.version_header feat: add google.api_core.version_header Apr 21, 2024
@parthea parthea changed the title feat: add google.api_core.version_header feat: add google.api_core.gapic_v1.version_header Apr 21, 2024
@parthea parthea changed the title feat: add google.api_core.gapic_v1.version_header feat: add google.api_core.version_header Apr 21, 2024
@parthea parthea force-pushed the add-api-version-header branch from 93ef5e9 to 7b84e34 Compare April 27, 2024 15:21
@parthea parthea marked this pull request as ready for review April 29, 2024 14:36
@parthea parthea requested review from a team as code owners April 29, 2024 14:36
Copy link
Contributor

@vchudnov-g vchudnov-g left a comment

Choose a reason for hiding this comment

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

Thanks for doing this!



def to_api_version_header(version_identifier):
"""Returns the API Version header for the given `version_identifier`.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: it's not the header itself, but the data for the header....

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in 60b33e6

@parthea parthea enabled auto-merge (squash) April 29, 2024 20:41
@parthea parthea merged commit a7b53e9 into main Apr 29, 2024
@parthea parthea deleted the add-api-version-header branch April 29, 2024 20:41
@parthea
Copy link
Collaborator Author

parthea commented Apr 29, 2024

/cherry-pick v1

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Apr 29, 2024
* feat: add google.api_core.gapic_v1.version_header

* fix mypy

* clarify comment
This was referenced May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants