Skip to content

feat(google/cloud/bigquery/v2): onboard a new library#8862

Closed
quirogas wants to merge 1 commit into
googleapis:mainfrom
quirogas:feat/onboard-bigquery-api
Closed

feat(google/cloud/bigquery/v2): onboard a new library#8862
quirogas wants to merge 1 commit into
googleapis:mainfrom
quirogas:feat/onboard-bigquery-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 adds the google-cloud-bigquery-api package, including the necessary configuration files, documentation, and proto definitions. The reviewer identified a version mismatch between librarian.yaml and package.json and suggested using metadata_name_override and name_pretty_override in librarian.yaml to properly handle naming conflicts with the existing bigquery library instead of manually editing the generated metadata file.

Comment thread librarian.yaml Outdated
Comment thread librarian.yaml
@quirogas
quirogas force-pushed the feat/onboard-bigquery-api branch 2 times, most recently from 69ffa22 to 34920bc Compare July 20, 2026 18:27
@quirogas
quirogas force-pushed the feat/onboard-bigquery-api branch from 34920bc to b05e639 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 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

We're going to close this one for now, because the BQ library is still using apiary rather than gapic. BQ storage is separate.

@feywind feywind closed this 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