Skip to content

Commit

Permalink
Merge pull request #1247 from opensafely-core/steve/replace-use-of-se…
Browse files Browse the repository at this point in the history
…t-output

Replace use of `set-output` in workflow
  • Loading branch information
StevenMaude committed May 11, 2023
2 parents 19c8335 + f7ecb3f commit 31258f8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build-and-deploy.yml
Expand Up @@ -74,12 +74,12 @@ jobs:
id: previoustagcommit
# if we are on a tag, should look like `v0.6.0`
# if we are not on a tag, should look like `v0.6.0-24-gbf6352d`
run: echo ::set-output name=tag_describe::`git describe --tags`
run: echo "tag_describe=`git describe --tags`" >> "$GITHUB_OUTPUT"

- name: Verify whether we are on a tagged version commit, allows us to short-circuit this workflow
id: taggedcommit
if: ${{ steps.previoustagcommit.outputs.tag_describe == steps.latest-version-tag.outputs.result }}
run: echo ::set-output name=tag::'y'
run: echo "tag=y" >> "$GITHUB_OUTPUT"

# Verify that 'Get latest version tag' returned a version tag
- name: Fail if no version tag found
Expand Down

0 comments on commit 31258f8

Please sign in to comment.