Skip to content

ci: add librarian library generation workflow#13284

Merged
sofisl merged 7 commits into
mainfrom
add-librarian-workflow
May 29, 2026
Merged

ci: add librarian library generation workflow#13284
sofisl merged 7 commits into
mainfrom
add-librarian-workflow

Conversation

@sofisl
Copy link
Copy Markdown
Contributor

@sofisl sofisl commented May 27, 2026

Create a new GitHub workflow librarian_library_generation.yaml to run librarian generate --all on PRs and verify that the output is identical to the checked-in code.

For googleapis/librarian#5240

Create a new GitHub workflow librarian_library_generation.yaml to run
librarian generate --all on PRs and verify that the output is identical
to the checked-in code.

Fixes googleapis/librarian#5240
@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.

Comment thread .github/workflows/librarian_library_generation.yaml Outdated
Comment thread .github/workflows/librarian_library_generation.yaml Outdated
sofisl added a commit that referenced this pull request May 28, 2026
In #13236, a
formatting change caused a diff between what is generated by librarian
and what actually exists. This blocks
#13284, because
there is a diff. This PR fixes the formatting issue. Then once we merge
that presubmit check, we can make sure these changes don't occur ad hoc.
@sofisl sofisl requested a review from zhumin8 May 28, 2026 22:08
@sofisl sofisl marked this pull request as ready for review May 28, 2026 22:08
@sofisl sofisl requested review from a team as code owners May 28, 2026 22:08
Comment thread .github/workflows/librarian_generation_check.yaml Outdated
Comment thread .github/workflows/librarian_generation_check.yaml Outdated
@sofisl sofisl requested review from blakeli0 and zhumin8 May 29, 2026 17:44
@sofisl sofisl enabled auto-merge (squash) May 29, 2026 17:58
# limitations under the License.
name: Librarian generate diff check on pull requests
on:
pull_request:
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.

How long does it take to run this workflow. If it's too long, e.g., > 10 mins, it may not be suitable for a presubmit.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Definitely longer than 10 min, closer to an hour.

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.

Definitely longer than 10 min, closer to an hour.

How about converting it to a postsubmit?

@sofisl sofisl merged commit 928140c into main May 29, 2026
186 of 187 checks passed
@sofisl sofisl deleted the add-librarian-workflow branch May 29, 2026 18:41
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.

4 participants