Skip to content

chore: add a post-submit check that regenerating doesn't change anything#16941

Merged
jskeet merged 2 commits intogoogleapis:mainfrom
jskeet:generate-on-push
May 6, 2026
Merged

chore: add a post-submit check that regenerating doesn't change anything#16941
jskeet merged 2 commits intogoogleapis:mainfrom
jskeet:generate-on-push

Conversation

@jskeet
Copy link
Copy Markdown
Contributor

@jskeet jskeet commented May 5, 2026

This is a post-submit as it takes a while (~25 minutes). We could potentially make it a pre-submit and only generate packages which have changed, but that would be more fiddly.

It would be nice not to specify the protoc or pandoc version here, but that will require improvements in librarian (tracked in googleapis/librarian#5818).

Fixes googleapis/librarian#4800

This is a post-submit as it takes a while (~25 minutes). We could
*potentially* make it a pre-submit and only generate packages which
have changed, but that would be more fiddly.

It would be nice not to specify the protoc or pandoc version here, but
that will require improvements in librarian (tracked in
googleapis/librarian#5818).

Fixes googleapis/librarian#4800
@jskeet jskeet requested a review from parthea May 5, 2026 12:09
@jskeet jskeet requested a review from a team as a code owner May 5, 2026 12:09
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@jskeet
Copy link
Copy Markdown
Contributor Author

jskeet commented May 6, 2026

Note: if we land googleapis/librarian#5860, the time may be significantly reduced. (It'll be hard to say by how much until it's in a released version of librarian, but it may end up taking less than 5 minutes.)

At that point, we could consider making this a presubmit instead of a postsubmit check.

Copy link
Copy Markdown
Contributor

@parthea parthea left a comment

Choose a reason for hiding this comment

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

nit: specify permissions for workflow

permissions:
  contents: read

@jskeet
Copy link
Copy Markdown
Contributor Author

jskeet commented May 6, 2026

permissions:
contents: read

Done, with issues: write as well.

@jskeet jskeet enabled auto-merge (squash) May 6, 2026 15:07
@jskeet jskeet merged commit 6544972 into googleapis:main May 6, 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.

google-cloud-python: check for code generation drift in CI

2 participants