Skip to content

chore: allow docs warnings in more packages#16716

Merged
jskeet merged 3 commits into
googleapis:mainfrom
jskeet:docs-warnings
Apr 20, 2026
Merged

chore: allow docs warnings in more packages#16716
jskeet merged 3 commits into
googleapis:mainfrom
jskeet:docs-warnings

Conversation

@jskeet

@jskeet jskeet commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Renames the post-processing file, and adds more paths to it.

This file should be removed entirely once we've upgraded to a later version of the GAPIC generator which fixes the warnings. See #15475 for details on this work.

Renames the post-processing file, and adds more paths to it.

This file should be removed entirely once we've upgraded to a later
version of the GAPIC generator which fixes the warnings. See googleapis#15475
for details on this work.
@jskeet jskeet requested a review from parthea April 20, 2026 06:56
@jskeet jskeet requested a review from a team as a code owner April 20, 2026 06:56

@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 updates the allow-docs-warnings.yaml configuration to suppress documentation warnings as errors for several Google Cloud Python packages, including compute, container, and dataplex. The review feedback suggests including missing compute-related packages (alpha and v1) for consistency and recommends sorting the package list alphabetically to improve maintainability.

@jskeet jskeet merged commit 39c50ca into googleapis:main Apr 20, 2026
30 checks passed
noahdietz added a commit that referenced this pull request Apr 29, 2026
Similar to #16716 disable warnings as errors for this one API to unblock
generation.
@jskeet jskeet deleted the docs-warnings branch May 13, 2026 07:44
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