Skip to content

Commit 390b696

Browse files
committed
workflows: test: Set permissions at workflow level
Permissions in job are probably cleaner, but since we're mainly including this workflow from others, it's easier to find the intended permissions in this way. Plus other workflows currently also manage their permissions at the workflow rather than job level. Also update permissions to more complete list. Signed-off-by: Loïc Minier <[email protected]>
1 parent ce82da5 commit 390b696

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

.github/workflows/test.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,12 @@ on:
77
required: true
88
type: string
99

10+
permissions:
11+
checks: write # EnricoMi/publish-unit-test-result-action
12+
pull-requests: write # EnricoMi/publish-unit-test-result-action
13+
contents: read # actions/checkout
14+
packages: read # actions/download-artifact
15+
1016
jobs:
1117
prepare-job-list:
1218
runs-on: ubuntu-latest
@@ -74,9 +80,6 @@ jobs:
7480
name: "Publish Tests Results"
7581
needs: submit-job
7682
runs-on: ubuntu-latest
77-
permissions:
78-
checks: write
79-
pull-requests: write
8083

8184
steps:
8285
- name: Download Artifacts

0 commit comments

Comments
 (0)