Sitelet https://github.com/googleapis/google-cloud-node-core/pull/788
Skip to content
This repository was archived by the owner on Mar 19, 2026. It is now read-only.

fix: add per-service apiVersion to gapic_metadata - #788

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

noahdietz merged 1 commit into
mainfrom
gapic-metadata-api-version

Conversation

@noahdietz

Copy link
Copy Markdown
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 subscribe to this conversation on GitHub. Already have an account? Sign in.

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