FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: fix release tag workflow to fetch all tags first by alicejli · Pull Request #1700 · googleapis/sdk-platform-java · GitHub

This repository was archived by the owner on May 14, 2026. It is now read-only.

fix: fix release tag workflow to fetch all tags first - #1700

Merged
alicejli merged 1 commit into
mainfrom
fixWorkflow
May 24, 2023
Merged

fix: fix release tag workflow to fetch all tags first#1700
alicejli merged 1 commit into
mainfrom
fixWorkflow

Conversation

Copy link
Copy Markdown
Contributor

Hopefully final follow-up to: #1699

Workflow currently fails if tag is already created. This is because the actions/checkout checks out the code in a fresh workspace that does not contain tags by default. This adds in the step of fetching tags.

product-auto-label Bot added the size: xs Pull request size is extra small. label May 23, 2023

suztomo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Alternatively we can make it override old tags. Either handling is fine, because I don't expect this workflow to run against an old release.

suztomo added the owlbot:run Add this label to trigger the Owlbot post processor. label May 24, 2023
gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 24, 2023
suztomo added automerge Merge the pull request once unit tests and other checks pass. owlbot:run Add this label to trigger the Owlbot post processor. labels May 24, 2023
gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label May 24, 2023

suztomo commented May 24, 2023

Copy link
Copy Markdown
Member

@mpeddada1 This change doesn't affect the release. This is just an error handling in case there's existing tags.

gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label May 24, 2023

Copy link
Copy Markdown

[gapic-generator-java-root] Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

alicejli marked this pull request as ready for review May 24, 2023 14:22
alicejli requested a review from a team May 24, 2023 14:22
alicejli merged commit 99b0c96 into main May 24, 2023
alicejli deleted the fixWorkflow branch May 24, 2023 14:23
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL