ci: do not checkout head ref for fork PRs, check against target of PR (…

…#344)
morremeyer · Jul 7, 2023 · 2f5ba0b · 2f5ba0b
1 parent 612e92c
commit 2f5ba0b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/.github/workflows/helm-docs-skip.yml b/.github/workflows/helm-docs-skip.yml
@@ -6,6 +6,6 @@ on:
 jobs:
   helm-docs:
     runs-on: ubuntu-latest
-    if: ${{ github.event.repository.name != 'charts' }}
+    if: ${{ github.event.pull_request.head.repo.full_name != 'morremeyer/charts' }}
     steps:
       - run: echo "Not a renovate PR, nothing to do"
diff --git a/.github/workflows/helm-docs.yml b/.github/workflows/helm-docs.yml
@@ -11,11 +11,9 @@ jobs:
     runs-on: ubuntu-latest
     # Only run this for PRs from the charts repo itself (to auto-update docs for renovate PRs)
     # For PRs made by humans, running pre-commit locally is required.
-    if: ${{ github.event.repository.name == 'charts' }}
+    if: ${{ github.event.pull_request.head.repo.full_name == 'morremeyer/charts' }}
     steps:
       - uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9 # v3
-        with:
-          ref: ${{ github.head_ref }}
 
       - uses: actions/setup-go@fac708d6674e30b6ba41289acaab6d4b75aa0753 # v4
         with: