Skip to content

chore: update hermetic build workflow#11114

Merged
JoeWang1127 merged 1 commit intomainfrom
chore/update-gen-workflow
Sep 3, 2024
Merged

chore: update hermetic build workflow#11114
JoeWang1127 merged 1 commit intomainfrom
chore/update-gen-workflow

Conversation

@JoeWang1127
Copy link
Contributor

In this PR:

This workflow will serve as a source of true for generation workflows in handwritten libraries.

@JoeWang1127 JoeWang1127 marked this pull request as ready for review September 3, 2024 17:55
@JoeWang1127 JoeWang1127 requested review from a team, blakeli0 and diegomarquezp September 3, 2024 17:55
Copy link
Contributor

@diegomarquezp diegomarquezp left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@JoeWang1127 JoeWang1127 enabled auto-merge (squash) September 3, 2024 18:10

if [ -z "${generation_config}" ]; then
generation_config=generation_config.yaml
echo "Use default generation config: ${generation_config}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we keep the image tag as a parameter? In long term, we do want to pass it in and remove the generator version from the generation_config.yaml.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry I missed this comment. I'll create a follow up.

@JoeWang1127 JoeWang1127 disabled auto-merge September 3, 2024 18:20
@JoeWang1127 JoeWang1127 merged commit 05713a3 into main Sep 3, 2024
@JoeWang1127 JoeWang1127 deleted the chore/update-gen-workflow branch September 3, 2024 18:20

# commit the change to the pull request.
git add java-* pom.xml gapic-libraries-bom/pom.xml versions.txt
if [[ $(basename "$(pwd)") == "google-cloud-java" ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have to differentiate between google-cloud-java and java-* repos? Can we just do git add .?

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.

3 participants