Skip to content

chore: create workflow to check showcase commit#13305

Merged
sofisl merged 13 commits into
mainfrom
showcase-version-check-workflow
Jun 2, 2026
Merged

chore: create workflow to check showcase commit#13305
sofisl merged 13 commits into
mainfrom
showcase-version-check-workflow

Conversation

@sofisl

@sofisl sofisl commented May 29, 2026

Copy link
Copy Markdown
Contributor

A workflow to make sure the showcase commit is the same as what is in librarian. If the workflow fails, or there is a mismatch, an issue will be created.

Fixes googleapis/librarian#6200

sofisl added 6 commits May 29, 2026 18:16
A Go script and workflow are added to check that the showcase commit in librarian.yaml is in sync with the gapic-showcase.version property in java-showcase/gapic-showcase/pom.xml.

Fixes googleapis/librarian#6200
A Go script and workflow are added to check that the showcase commit in librarian.yaml is in sync with the gapic-showcase.version property in java-showcase/gapic-showcase/pom.xml.

Fixes googleapis/librarian#6200

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request introduces a new Go script, .github/scripts/verify_showcase_version.go, to verify that the showcase version and commit are in sync by querying the remote repository using git ls-remote. A review comment identifies an issue where annotated tags in Git produce multiple lines in the git ls-remote output, which can cause the script to incorrectly parse the tag object's hash instead of the actual commit hash. A code suggestion is provided to parse the output line-by-line to correctly identify the dereferenced commit.

Comment thread .github/scripts/verify_showcase_version.go Outdated
@sofisl

sofisl commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

The logic is pretty complicated so I think maybe putting it all in a go file would be better, but I'm taking from this that maybe you'd prefer some more bash scripting.

sofisl and others added 2 commits May 29, 2026 11:52
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@sofisl sofisl marked this pull request as ready for review May 29, 2026 19:33
@sofisl sofisl requested review from a team as code owners May 29, 2026 19:33
Comment thread .github/workflows/showcase-version-check.yaml
Comment thread .github/workflows/showcase-version-check.yaml Outdated
@sofisl sofisl requested a review from zhumin8 May 29, 2026 19:47
Comment thread .github/scripts/verify_showcase_version.go Outdated

@zhumin8 zhumin8 left a comment

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.

Merge with main you should be able to avoid the irrelevant ci/unit tests (#13299).

@sofisl sofisl enabled auto-merge (squash) June 2, 2026 17:13
@sofisl sofisl merged commit c3dd3d7 into main Jun 2, 2026
147 of 149 checks passed
@sofisl sofisl deleted the showcase-version-check-workflow branch June 2, 2026 17:14
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-java: create workflow check for showcase commit used

2 participants