Skip to content

feat(google/logging/v2): onboard a new library#8863

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

feat(google/logging/v2): onboard a new library#8863
feywind merged 1 commit into
googleapis:mainfrom
quirogas:feat/onboard-logging-api

Conversation

@quirogas

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/logging-api package, which contains auto-generated clients, samples, and proto definitions for the Cloud Logging API (including Config, Logging, and Metrics services). The review feedback highlights two issues: first, the auto-generated links in the README point to the wrong package directory and should be corrected upstream in the generator templates; second, the webpack configuration should be updated to use a more generic library name and filename, as the package exports multiple services rather than just ConfigServiceV2.

Comment thread packages/google-cloud-logging-api/README.md Outdated
Comment thread packages/google-cloud-logging-api/webpack.config.js
@quirogas
quirogas force-pushed the feat/onboard-logging-api branch 2 times, most recently from 2be3e15 to 9be6a9b Compare July 20, 2026 18:27
@quirogas quirogas self-assigned this Jul 20, 2026
@quirogas
quirogas force-pushed the feat/onboard-logging-api branch from 9be6a9b to 93e9cd5 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
Comment thread packages/google-cloud-logging-api/README.md
@quirogas
quirogas force-pushed the feat/onboard-logging-api branch from 93e9cd5 to 5fb8c11 Compare July 20, 2026 20:14
@quirogas
quirogas requested a review from feywind July 20, 2026 20:25
@feywind
feywind merged commit 2ac07e4 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