Skip to content

Commit

Permalink
Merge pull request #17 from eapolinario/fix-image-version
Browse files Browse the repository at this point in the history
Fix piping of next version
  • Loading branch information
eapolinario authored Sep 26, 2023
2 parents 0e9f89e + e35e23f commit 8db7d8f
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
9 changes: 4 additions & 5 deletions .github/workflows/bump-tags.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@ name: Bump git tags
on:
workflow_call:
outputs:
version:
next_version:
description: "Tag name"
value: ${{ jobs.bump_version.outputs.version }}
value: ${{ jobs.bump_version.outputs.next_version }}
secrets:
FLYTE_BOT_PAT:
required: true
Expand All @@ -14,7 +14,7 @@ jobs:
name: Bump Version
runs-on: ubuntu-latest
outputs:
version: ${{ steps.bump_version.outputs.tag }}
next_version: ${{ steps.get_semver.outputs.next_version }}
steps:
- uses: actions/checkout@v3
with:
Expand All @@ -24,8 +24,7 @@ jobs:
id: get_semver
with:
bump_level: "patch"

- name: Print current and next version
- name: Print current and next versions
run: |
echo "Current version: ${{ steps.get_semver.outputs.current_version }}"
echo "Next version: ${{ steps.get_semver.outputs.next_version }}"
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ jobs:
- flytescheduler
uses: ./.github/workflows/publish.yml
with:
version: ${{ needs.bump-tags.outputs.version }}
version: ${{ needs.bump-tags.outputs.next_version }}
component: ${{ matrix.component }}
dockerfile: Dockerfile.${{ matrix.component }}
push: true
Expand Down

0 comments on commit 8db7d8f

Please sign in to comment.