Do not run report action on forks to avoid permissions issues

firedrakeproject · Jan 15, 2025 · e608276 · e608276
1 parent ad9fe2c
commit e608276
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -109,7 +109,9 @@ jobs:
         timeout-minutes: 120
       - name: Publish Test Report
         uses: mikepenz/[email protected]
-        if: ${{ always() && ( github.ref != 'refs/heads/master') }}
+        # To avoid permissions issues do not run with forked repos
+        # (see https://github.com/mikepenz/action-junit-report/issues/23)
+        if: ${{ always() && (github.ref != 'refs/heads/master') && (github.event.pull_request.head.repo.full_name == github.repository) }}
         with:
           report_paths: '/__w/firedrake/firedrake/firedrake.xml'
           comment: true

diff --git a/.github/workflows/pip.yml b/.github/workflows/pip.yml
@@ -96,7 +96,9 @@ jobs:
 
       - name: Publish Test Report
         uses: mikepenz/[email protected]
-        if: ${{ always() && ( github.ref != 'refs/heads/master') }}
+        # To avoid permissions issues do not run with forked repos
+        # (see https://github.com/mikepenz/action-junit-report/issues/23)
+        if: ${{ always() && (github.ref != 'refs/heads/master') && (github.event.pull_request.head.repo.full_name == github.repository) }}
         with:
           report_paths: '/home/firedrake/pip_venv/src/firedrake/firedrake.xml'
           comment: true