FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: add google.api_core.version_header by parthea · Pull Request #638 · googleapis/python-api-core · GitHub

This repository was archived by the owner on Feb 23, 2026. It is now read-only.
/ python-api-core Public archive

feat: add google.api_core.version_header - #638

Merged
parthea merged 4 commits into
mainfrom
add-api-version-header
Apr 29, 2024
Merged

feat: add google.api_core.version_header#638
parthea merged 4 commits into
mainfrom
add-api-version-header

Conversation

parthea commented Apr 8, 2024

Copy link
Copy Markdown
Contributor

Towards b/330968465

product-auto-label Bot added the size: xs Pull request size is extra small. label Apr 8, 2024
parthea force-pushed the add-api-version-header branch from c4498a7 to cbe0bb1 Compare April 8, 2024 16:05
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 force-pushed the add-api-version-header branch 4 times, most recently from 578b720 to ad08619 Compare April 8, 2024 20:50
parthea changed the title feat: add google.api_core.gapic_v1.version_header feat: add google.api_core.version_header Apr 21, 2024
parthea changed the title feat: add google.api_core.version_header feat: add google.api_core.gapic_v1.version_header Apr 21, 2024
parthea changed the title feat: add google.api_core.gapic_v1.version_header feat: add google.api_core.version_header Apr 21, 2024
parthea force-pushed the add-api-version-header branch from 93ef5e9 to 7b84e34 Compare April 27, 2024 15:21
parthea marked this pull request as ready for review April 29, 2024 14:36
parthea requested review from a team April 29, 2024 14:36

vchudnov-g 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks for doing this!

Comment thread google/api_core/version_header.py Outdated


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

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Fixed in 60b33e6

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

parthea commented Apr 29, 2024

Copy link
Copy Markdown
Contributor Author

/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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL