Skip to content

feat: API configuration: 20260512T090125Z#15615

Merged
amanda-tarafa merged 1 commit into
mainfrom
librarian-config-20260512T090125Z
May 12, 2026
Merged

feat: API configuration: 20260512T090125Z#15615
amanda-tarafa merged 1 commit into
mainfrom
librarian-config-20260512T090125Z

Conversation

@google-cloud-sdk-librarian-dotnet-robot
Copy link
Copy Markdown
Collaborator

Changes in this PR

  • Configured library Google.Backstory for API backstory

@github-actions
Copy link
Copy Markdown

Pull request diff results
No APIs have changed in this PR. Exiting diff.

@jskeet jskeet requested a review from amanda-tarafa May 12, 2026 09:09
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 the configuration for the Google.Backstory API to the generator input and pipeline state. The review feedback identifies two consistency and branding issues: the use of an internal codename in the product name and a deviation from the standard Google.Cloud naming convention for the API ID.

Comment thread generator-input/apis.json
"id": "Google.Backstory",
"version": "1.0.0-beta00",
"type": "grpc",
"productName": "Malachite Common Protos",
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 productName "Malachite Common Protos" uses an internal codename ("Malachite"). This should be updated to a public-facing name such as "Chronicle Common Protos" or "Google Security Operations Common Protos" to align with the product's public branding and the provided productUrl.

Suggested change
"productName": "Malachite Common Protos",
"productName": "Chronicle Common Protos",

Comment thread generator-input/apis.json
"includeCommonResourcesProto": true
},
{
"id": "Google.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 id "Google.Backstory" deviates from the standard Google.Cloud.{Service} naming convention used for other libraries in this configuration (e.g., Google.Cloud.AccessApproval.V1). To maintain consistency with the repository's naming standards for Cloud APIs, consider using Google.Cloud.Backstory.

@amanda-tarafa amanda-tarafa merged commit 493e82e into main May 12, 2026
14 checks passed
@amanda-tarafa amanda-tarafa deleted the librarian-config-20260512T090125Z branch May 12, 2026 21:05
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