Skip to content

feat(google/firestore/v1): onboard a new library#8859

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

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

Conversation

@quirogas

Copy link
Copy Markdown
Contributor

No description provided.

@quirogas quirogas self-assigned this Jul 10, 2026

@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 a new package, @google-cloud/firestore-api, containing the auto-generated Node.js client for both the Cloud Firestore data and admin APIs. The review feedback highlights that several files (including package.json, README.md, and webpack.config.js) incorrectly describe the package as solely an admin client or contain incorrect links. The reviewer recommends updating the upstream generator templates for the auto-generated files and provides specific code suggestions to correct the package description and webpack library name.

Comment thread packages/google-cloud-firestore-api/package.json Outdated
Comment thread packages/google-cloud-firestore-api/README.md Outdated
Comment thread packages/google-cloud-firestore-api/README.md Outdated
Comment thread packages/google-cloud-firestore-api/webpack.config.js
@quirogas
quirogas force-pushed the feat/onboard-firestore-api branch 3 times, most recently from b59683a to ba61375 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.

Looks good besides the README.

@quirogas
quirogas force-pushed the feat/onboard-firestore-api branch from ba61375 to 6e6a41b Compare July 20, 2026 20:15
@quirogas
quirogas requested a review from feywind July 20, 2026 20:25
@feywind
feywind merged commit e39cdbd 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