Skip to content

fix: get-release step runs when upload-to-release is false - #91

Open
andrewwhitecdw wants to merge 1 commit into
NVlabs:mainfrom
andrewwhitecdw:bugfix/build-06dd9993
Open

fix: get-release step runs when upload-to-release is false#91
andrewwhitecdw wants to merge 1 commit into
NVlabs:mainfrom
andrewwhitecdw:bugfix/build-06dd9993

Conversation

@andrewwhitecdw

Copy link
Copy Markdown

This PR addresses the following issue in fouroversix/.github/workflows/_build.yml: get-release step runs when upload-to-release is false.

Changes

  • fouroversix/.github/workflows/_build.yml: get-release step runs when upload-to-release is false.

Details

--- a/fouroversix/.github/workflows/_build.yml
+++ b/fouroversix/.github/workflows/_build.yml
@@ -1,7 +1,8 @@
-      - name: Get Release with tag
-        id: get_current_release
-        uses: joutvhu/get-release@v1
-        with:
-          tag_name: ${{ inputs.release-version }}
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      - name: Get Release with tag
+        if: inputs.upload-to-release
+        id: get_current_release
+        uses: joutvhu/get-release@v1
+        with:
+          tag_name: ${{ inputs.release-version }}
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Tests

Let me know if you want tests added for this fix or not.

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.

1 participant