Skip to content

chore: skip generation for packages we need to update individually#17127

Merged
jskeet merged 1 commit into
googleapis:mainfrom
jskeet:skip-generation
May 14, 2026
Merged

chore: skip generation for packages we need to update individually#17127
jskeet merged 1 commit into
googleapis:mainfrom
jskeet:skip-generation

Conversation

@jskeet
Copy link
Copy Markdown
Contributor

@jskeet jskeet commented May 14, 2026

These packages have not yet been updated to generator version 1.32.0. We should handle them with a separate PR per package which fixes any post-processing files, unskips generation, and includes the generated changes.

Packages skipped by this change:

  • google-cloud-asset
  • google-cloud-automl
  • google-cloud-monitoring
  • google-cloud-pubsub
  • google-cloud-spanner
  • google-cloud-storage

These packages have not yet been updated to generator version 1.32.0.
We should handle them with a separate PR per package which fixes any
post-processing files, unskips generation, and includes the generated
changes.

Packages skipped by this change:

- google-cloud-asset
- google-cloud-automl
- google-cloud-monitoring
- google-cloud-pubsub
- google-cloud-spanner
- google-cloud-storage
@jskeet jskeet requested a review from chalmerlowe May 14, 2026 06:51
@jskeet jskeet requested a review from a team as a code owner May 14, 2026 06:51
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 librarian.yaml configuration file to set the skip_generate flag to true for several libraries, including Cloud Asset, AutoML, Monitoring, Pub/Sub, Spanner, and Storage. I have no feedback to provide as there were no review comments.

Copy link
Copy Markdown
Contributor

@chalmerlowe chalmerlowe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jskeet jskeet merged commit e46cd6d into googleapis:main May 14, 2026
31 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