Skip to content

feat: Add backstory to chronicle API#13327

Draft
blakeli0 wants to merge 1 commit into
mainfrom
blakeli0-patch-12
Draft

feat: Add backstory to chronicle API#13327
blakeli0 wants to merge 1 commit into
mainfrom
blakeli0-patch-12

Conversation

@blakeli0
Copy link
Copy Markdown
Contributor

@blakeli0 blakeli0 commented Jun 1, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

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 a new proto path backstory to the GAPICs list in generation_config.yaml. The reviewer pointed out that this path appears to be incomplete and should likely follow a fully-qualified structured namespace, such as starting with google/cloud/ and ending with a version.

Comment thread generation_config.yaml
cloud_api: true
GAPICs:
- proto_path: google/cloud/chronicle/v1
- proto_path: backstory
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.

medium

The proto path backstory appears to be incomplete. Typically, proto paths in this configuration follow a structured namespace (e.g., starting with google/cloud/ and ending with a version like /v1). Please verify and update the path to the correct fully-qualified proto directory.

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.

1 participant