Skip to content

Commit 09218e0

Browse files
fix(ci): move write permissions to job level for Scorecard compliance
- pr-tests.yml: Set workflow-level to read-all, add explicit permissions to each job - unity-tests.yml: Set workflow-level to read-all, keep job-level write permissions This addresses OpenSSF Scorecard Token-Permissions alerts. Signed-off-by: JasonXuDeveloper <jasonxudeveloper@gmail.com> Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
1 parent 8f176a1 commit 09218e0

2 files changed

Lines changed: 6 additions & 7 deletions

File tree

.github/workflows/pr-tests.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,14 @@ concurrency:
1212
group: pr-tests-${{ github.event.pull_request.number }}
1313
cancel-in-progress: true
1414

15-
permissions:
16-
contents: read
17-
pull-requests: write
18-
statuses: write
15+
permissions: read-all
1916

2017
jobs:
2118
changes:
2219
name: Detect Changes
2320
runs-on: ubuntu-latest
21+
permissions:
22+
contents: read
2423
outputs:
2524
should_test: ${{ steps.filter.outputs.src }}
2625
steps:
@@ -75,6 +74,8 @@ jobs:
7574
needs: [changes, run-tests]
7675
runs-on: ubuntu-latest
7776
if: needs.changes.outputs.should_test == 'true' && needs.run-tests.result == 'success'
77+
permissions:
78+
contents: read
7879
steps:
7980
- name: Checkout repository
8081
uses: actions/checkout@v4

.github/workflows/unity-tests.yml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,7 @@ on:
2323
description: 'Test results summary'
2424
value: ${{ jobs.test.outputs.results }}
2525

26-
permissions:
27-
contents: read
28-
checks: write
26+
permissions: read-all
2927

3028
jobs:
3129
test:

0 commit comments

Comments
 (0)