Skip to content

Commit ed04bba

Browse files
committed
More precaution by checking if the action has tagged and pushed.
This will prevent unnecessary deletion of tags if the tagging was not successful but tag already exists.
1 parent 427098f commit ed04bba

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.github/workflows/create-release.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ jobs:
2828
tag_name: v${{ steps.read-version.outputs.version_full }}
2929
version_commit: ${{ steps.version_commit.outputs.commit_long_sha }}
3030
pushed: ${{ steps.version_commit.outputs.pushed }}
31+
tagged: ${{ steps.version_commit.outputs.tagged }}
3132
tag_pushed: ${{ steps.version_commit.outputs.tag_pushed }}
3233
if: github.event_name != 'schedule' || (github.event_name == 'schedule' && github.repository == 'quarto-dev/quarto-cli')
3334
steps:
@@ -656,7 +657,7 @@ jobs:
656657
git push origin $GITHUB_REF_NAME
657658
658659
- name: Deleted created tag
659-
if: ${{ needs.configure.outputs.tag_pushed }}
660+
if: ${{ needs.configure.outputs.tagged && needs.configure.outputs.tag_pushed }}
660661
run: |
661662
git push --delete origin ${{ needs.configure.outputs.tag_name }}
662663

0 commit comments

Comments
 (0)