Skip to content

chore: remove description_override (batch 1 - to google-cloud-apphub)#16885

Merged
parthea merged 3 commits intogoogleapis:mainfrom
jskeet:description-batch-1
Apr 30, 2026
Merged

chore: remove description_override (batch 1 - to google-cloud-apphub)#16885
parthea merged 3 commits intogoogleapis:mainfrom
jskeet:description-batch-1

Conversation

@jskeet
Copy link
Copy Markdown
Contributor

@jskeet jskeet commented Apr 30, 2026

No description provided.

@jskeet jskeet requested a review from parthea April 30, 2026 06:37
@jskeet jskeet requested review from a team as code owners April 30, 2026 06:37
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 removes description_override entries from librarian.yaml and updates api_description fields across several package metadata and README files to improve consistency. A review comment identifies the removal of the -W (warnings as errors) flag in packages/google-analytics-data/noxfile.py and suggests restoring it to ensure build configurations remain consistent across the repository.

session.run(
"sphinx-build",
"-W", # warnings as errors
"-T", # show full traceback on exception
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 -W flag (warnings as errors) was removed from the documentation build process. Changes to shared configurations, like build flags in noxfile.py, should be applied consistently across all relevant packages in a dedicated, universal change rather than piecemeal in individual pull requests.

Suggested change
"-T", # show full traceback on exception
"-W", # warnings as errors
"-T", # show full traceback on exception
References
  1. Changes to shared configurations, like mypy flags, should be applied consistently across all relevant packages in a dedicated, universal change rather than piecemeal in individual pull requests.

@parthea parthea merged commit 7bfff06 into googleapis:main Apr 30, 2026
29 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