Skip to content

Commit

Permalink
Fix duplicate yq in github actions for chart publishing (#7)
Browse files Browse the repository at this point in the history
* Add support for separating out platform config

* Address lint findings

* Remove duplicate yq directive from github actions
  • Loading branch information
andykrohg authored Feb 3, 2021
1 parent 7f8087d commit f098a07
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/publish-helm-charts-edge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ jobs:
uses: docker://mikefarah/yq
with:
args:
yq write --inplace charts/${{ matrix.helm-chart }}/Chart.yaml version ${{ steps.helm-chart-next-semver.outputs.next-semver }}
write --inplace charts/${{ matrix.helm-chart }}/Chart.yaml version ${{ steps.helm-chart-next-semver.outputs.next-semver }}

# NOTE: workaround for https://github.com/helm/chart-releaser/issues/95
- name: Create Chart.lock ⚙️
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/publish-helm-charts-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ jobs:
uses: docker://mikefarah/yq
with:
args:
yq write --inplace charts/${{ matrix.helm-chart }}/Chart.yaml version ${{ needs.setup.outputs.helm-charts-new-semver }}
write --inplace charts/${{ matrix.helm-chart }}/Chart.yaml version ${{ needs.setup.outputs.helm-charts-new-semver }}

# NOTE: workaround for https://github.com/helm/chart-releaser/issues/95
- name: Create Chart.lock ⚙️
Expand Down

0 comments on commit f098a07

Please sign in to comment.