Skip to content

chore: fix librarian regeneration check#16968

Merged
jskeet merged 1 commit intogoogleapis:mainfrom
jskeet:fix-regeneration-check
May 6, 2026
Merged

chore: fix librarian regeneration check#16968
jskeet merged 1 commit intogoogleapis:mainfrom
jskeet:fix-regeneration-check

Conversation

@jskeet
Copy link
Copy Markdown
Contributor

@jskeet jskeet commented May 6, 2026

There are two reasons we should create an issue:

  • Regeneration failed
  • Regeneration succeeded, but created a diff

The previous implementation was half of one, half of the other. This separates out the two cases, creating an appropriate issue in each case.

Fixes googleapis/librarian#5866

There are two reasons we should create an issue:
- Regeneration failed
- Regeneration succeeded, but created a diff

The previous implementation was half of one, half of the other. This
separates out the two cases, creating an appropriate issue in each case.

Fixes googleapis/librarian#5866
@jskeet jskeet requested a review from parthea May 6, 2026 17:32
@jskeet jskeet requested a review from a team as a code owner May 6, 2026 17:32
@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 jskeet merged commit 7b2983b 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-go: generation check does not actually check for diffs

2 participants