Skip to content

fix: add per-service apiVersion to gapic_metadata#788

Merged
noahdietz merged 1 commit intomainfrom
gapic-metadata-api-version
Nov 5, 2025
Merged

fix: add per-service apiVersion to gapic_metadata#788
noahdietz merged 1 commit intomainfrom
gapic-metadata-api-version

Conversation

@noahdietz
Copy link
Contributor

Populates the api_version field on a per-service basis to the generated gapic_metadata.json using the value from the service-level annotation google.api.api_version already parsed for use in the X-Goog-Api-Version/$apiVersion system parameter (as of googleapis/gapic-generator-typescript#1576).

Showcase-based tests already have google.api.api_version set on service Echo, so the baselines are appropriately updated to account for inclusion of apiVersion in the gapic_metadata.json output.

It's presence in gapic_metadata.json will allow parsers to correlate service client to API version. It is not in use by any API yet. Only the unit tests needed updating.

Internal bug http://b/452365074.

@noahdietz noahdietz added the automerge Merge the pull request once unit tests and other checks pass. label Nov 5, 2025
@noahdietz noahdietz merged commit 506e3ef into main Nov 5, 2025
12 checks passed
@noahdietz noahdietz deleted the gapic-metadata-api-version branch November 5, 2025 23:08
This was referenced Nov 6, 2025
miguelvelezsa pushed a commit that referenced this pull request Nov 6, 2025
* updated CHANGELOG.md [ci skip]

* updated package.json [ci skip]

* updated samples/package.json [ci skip]
@release-please release-please bot mentioned this pull request Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the pull request once unit tests and other checks pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants