Fixed permission clash on workflow reuse

Open-CMSIS-Pack · Nov 29, 2023 · 0900942 · 0900942
1 parent 43f1e21
commit 0900942
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -5,9 +5,6 @@ on:
     tags:
       - "v*"
 
-# Declare default permissions as read only.
-permissions: read-all
-
 jobs:
   test:
     uses: Open-CMSIS-Pack/cbuild/.github/workflows/test.yml@main

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -125,7 +125,7 @@ jobs:
           if-no-files-found: error
 
   publish-test-results:
-    if: github.workflow != 'Release'
+    if: ${{ github.workflow != 'Release' }}
     name: "Publish Tests Results"
     needs: [ test ]
     runs-on: ubuntu-latest
@@ -145,7 +145,7 @@ jobs:
           report_individual_runs: true
 
   coverage:
-    if: github.workflow != 'Release'
+    if: ${{ github.workflow != 'Release' && github.repository == 'Open-CMSIS-Pack/cbuild' }}
     needs: [ test ]
     name: 'Coverage check'
     runs-on: ubuntu-latest
-Original file line number
+Diff line change
@@ Expand Up / @@ -5,9 +5,6 @@ on: @@
         tags:
           - "v*"
-    # Declare default permissions as read only.
-    permissions: read-all
     jobs:
       test:
         uses: Open-CMSIS-Pack/cbuild/.github/workflows/test.yml@main
@@ Expand Down @@