Skip to content

feat(google/spanner/v1): onboard a new library#8860

Merged
feywind merged 1 commit into
googleapis:mainfrom
quirogas:feat/onboard-spanner-api
Jul 20, 2026
Merged

feat(google/spanner/v1): onboard a new library#8860
feywind merged 1 commit into
googleapis:mainfrom
quirogas:feat/onboard-spanner-api

Conversation

@quirogas

@quirogas quirogas commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@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 introduces the new @google-cloud/spanner-api package, which provides auto-generated Node.js clients for Cloud Spanner, including database administration, instance administration, and core Spanner services. Feedback on the changes highlights that the generated instance_admin_client.ts file is missing from the PR, and the entrypoint files currently only export InstanceAdminClient instead of all generated clients. Additionally, it is recommended to update the package description in package.json to be more generic and accurate.

Comment thread packages/google-cloud-spanner-api/src/v1/index.ts
Comment thread packages/google-cloud-spanner-api/src/index.ts Outdated
Comment thread packages/google-cloud-spanner-api/package.json Outdated
@quirogas
quirogas force-pushed the feat/onboard-spanner-api branch 2 times, most recently from 3a2150d to 088fa22 Compare July 20, 2026 18:28
@quirogas quirogas self-assigned this Jul 20, 2026
@quirogas
quirogas force-pushed the feat/onboard-spanner-api branch from 088fa22 to 0c77488 Compare July 20, 2026 18:36
@quirogas
quirogas marked this pull request as ready for review July 20, 2026 18:58
@quirogas
quirogas requested a review from a team as a code owner July 20, 2026 18:58

@feywind feywind 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.

Missing the README but otherwise looks good.

@quirogas
quirogas force-pushed the feat/onboard-spanner-api branch from 0c77488 to 4c7e035 Compare July 20, 2026 20:15
@quirogas
quirogas requested a review from feywind July 20, 2026 20:25
@feywind
feywind merged commit b01b02d into googleapis:main Jul 20, 2026
34 checks passed
@release-please release-please Bot mentioned this pull request Jul 20, 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