Skip to content

Commit

Permalink
Merge pull request #32 from mozilla-it/fix-diff-rendered-charts
Browse files Browse the repository at this point in the history
fix(diff-rendered-charts): using environment variable for head_ref
  • Loading branch information
dlactin authored May 28, 2024
2 parents 4fbdc69 + 8bdc403 commit 33a6e06
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/diff-rendered-charts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ name: render and diff helm charts
on:
workflow_call:

env:
HEAD_REF: ${{ github.head_ref }}

jobs:
get_changed_helm_charts:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -45,8 +48,8 @@ jobs:
id: render_head
run: |
mkdir -p shared/head-charts
git fetch origin ${{ github.head_ref }}
git checkout ${{ github.head_ref }} --
git fetch origin "$HEAD_REF"
git checkout "$HEAD_REF" --
if [ -f "${{ matrix.chart }}/Chart.yaml" ]; then
helm dependency update "${{ matrix.chart }}"
values_files="${{ matrix.chart }}"/values-*
Expand Down

0 comments on commit 33a6e06

Please sign in to comment.