Skip to content

chore: remove name_pretty_override (batch 3)#16871

Merged
jskeet merged 2 commits intogoogleapis:mainfrom
jskeet:remove-name-pretty-override-8
Apr 29, 2026
Merged

chore: remove name_pretty_override (batch 3)#16871
jskeet merged 2 commits intogoogleapis:mainfrom
jskeet:remove-name-pretty-override-8

Conversation

@jskeet
Copy link
Copy Markdown
Contributor

@jskeet jskeet commented Apr 29, 2026

This batch is Cloud APIs up to but not including KMS.

jskeet added 2 commits April 29, 2026 13:56
This batch is Cloud APIs up to but not including KMS.
@jskeet jskeet requested a review from parthea April 29, 2026 14:36
@jskeet jskeet requested review from a team as code owners April 29, 2026 14:36
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 updates the naming conventions for various Google Cloud client libraries by removing redundant 'API' suffixes from product names in metadata files, READMEs, and documentation summaries. The changes ensure consistency across the documentation. All review comments were identified as valid, as they point out redundant 'API' suffixes in the summary documentation that should be removed to align with the new naming standards.

]: #

# Confidential Computing API API
# Confidential Computing API
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 title contains a redundant 'API' suffix after the removal of 'name_pretty_override'. Please remove the extra 'API' to match the updated naming convention.

]: #

# Infrastructure Manager API API
# Infrastructure Manager API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

]: #

# Config Delivery API API
# Config Delivery API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

]: #

# Database Center API API
# Database Center API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

]: #

# Data Lineage API API
# Data Lineage API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

]: #

# Cluster Director API API
# Cluster Director API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

]: #

# Cloud Identity and Access Management API
# Identity and Access Management (IAM) API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

]: #

# IAM Logging Protos API
# IAM Audit Logging API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

]: #

# iamconnectorcredentials.googleapis.com API API
# iamconnectorcredentials.googleapis.com API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

]: #

# Identity-Aware Proxy API
# Cloud Identity-Aware Proxy API
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 title contains a redundant 'API' suffix. Please remove the extra 'API' to match the updated naming convention.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

@jskeet jskeet merged commit ae048f0 into googleapis:main Apr 29, 2026
30 checks passed
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