Skip to content

chore: remove name_pretty_override (batch 5)#16874

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

chore: remove name_pretty_override (batch 5)#16874
jskeet merged 2 commits intogoogleapis:mainfrom
jskeet:remove-name-pretty-override-10

Conversation

@jskeet
Copy link
Copy Markdown
Contributor

@jskeet jskeet commented Apr 29, 2026

This removes the override from the remaining Cloud APIs.

@jskeet jskeet requested a review from parthea April 29, 2026 15:01
@jskeet jskeet requested review from a team as code owners April 29, 2026 15:01
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 standardizes the 'pretty names' for various Google Cloud Python client libraries by updating metadata and documentation across multiple packages, primarily to remove redundant 'API' suffixes. Feedback was provided regarding the continued use of the word 'API' in the titles of summary_overview.md files, which the reviewer noted as redundant and inconsistent with the naming conventions established in the metadata and README files of the same packages.

]: #

# SaaS Runtime API API
# App Lifecycle 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 includes the word 'API' redundantly, as the product name 'App Lifecycle Manager' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Secure Source Manager API API
# Secure Source 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 includes the word 'API' redundantly, as the product name 'Secure Source Manager' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Google Cloud Security Command Center API
# Security Command 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 includes the word 'API' redundantly, as the product name 'Security Command Center' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Security Center Management API API
# Security Command Center Management 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 includes the word 'API' redundantly, as the product name 'Security Command Center Management' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Service Health API API
# Service Health 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 includes the word 'API' redundantly, as the product name 'Service Health' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Google Cloud VMware Engine API
# VMware Engine 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 includes the word 'API' redundantly, as the product name 'VMware Engine' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Virtual Private Cloud API
# Serverless VPC Access 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 includes the word 'API' redundantly, as the product name 'Serverless VPC Access' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Cloud Security Scanner API
# Web Security Scanner 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 includes the word 'API' redundantly, as the product name 'Web Security Scanner' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Cloud Workflows API
# Workflows 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 includes the word 'API' redundantly, as the product name 'Workflows' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

]: #

# Workload Manager API API
# Workload 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 includes the word 'API' redundantly, as the product name 'Workload Manager' already implies it is an API. This is inconsistent with the naming convention used in other files in this PR.

@parthea
Copy link
Copy Markdown
Contributor

parthea commented Apr 29, 2026

The 2 presubmits are likely a Github issue

Build (369568738723)
Build (391947230478)

Rebasing this PR may clean these up

jskeet added 2 commits April 29, 2026 15:23
This removes the override from the remaining Cloud APIs.
@jskeet jskeet force-pushed the remove-name-pretty-override-10 branch from 14f0406 to 96db0d3 Compare April 29, 2026 15:23
@jskeet jskeet merged commit e8a3590 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