Skip to content

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

Open
jskeet wants to merge 1 commit intogoogleapis:mainfrom
jskeet:generate-on-push
Open

chore: add a post-submit check that regenerating doesn't change anything#16941
jskeet wants to merge 1 commit 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.

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

1 participant