Skip to content

Commit 7f67cf3

Browse files
committed
Split steps for minimal permissions
1 parent 4547b9c commit 7f67cf3

1 file changed

Lines changed: 29 additions & 6 deletions

File tree

.github/workflows/coverage_report.yml

Lines changed: 29 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,9 @@ on:
99
permissions: {}
1010

1111
jobs:
12-
cov-report:
13-
permissions:
14-
contents: read
15-
pull-requests: write
12+
get-pr-number:
1613
runs-on: ubuntu-latest
14+
permissions: {}
1715
if: >
1816
github.event.workflow_run.event == 'pull_request'
1917
&& github.repository == github.event.workflow_run.repository.full_name
@@ -35,10 +33,20 @@ jobs:
3533
fi
3634
PR_NUMBER=$(jq '.items[0].number' <<< "$PR_DATA")
3735
echo "pr_number=$PR_NUMBER" >> $GITHUB_OUTPUT
36+
outputs:
37+
pr_number: ${{ steps.get_pr_number.outputs.pr_number }}
38+
39+
create-md-comment:
40+
runs-on: ubuntu-latest
41+
permissions: {}
42+
if: >
43+
github.event.workflow_run.event == 'pull_request'
44+
&& github.repository == github.event.workflow_run.repository.full_name
45+
steps:
3846
- name: Download coverage report
3947
uses: actions/download-artifact@v4
4048
with:
41-
name: coverage.xml
49+
name: coverage
4250
github-token: ${{ secrets.GITHUB_TOKEN }}
4351
run-id: ${{ github.event.workflow_run.id }}
4452
- name: Code Coverage Report
@@ -53,10 +61,25 @@ jobs:
5361
indicators: true
5462
output: both
5563
thresholds: '80 90'
64+
- name: Upload coverage summary
65+
uses: actions/upload-artifact@v4
66+
with:
67+
name: code-coverage-results
68+
path: code-coverage-results.md
69+
post-comment:
70+
runs-on: ubuntu-latest
71+
needs: [get-pr-number, create-md-comment]
72+
permissions:
73+
pull-requests: write
74+
steps:
75+
- name: Download coverage report
76+
uses: actions/download-artifact@v4
77+
with:
78+
name: code-coverage-results
5679
- name: Add Coverage PR Comment
5780
uses: marocchino/sticky-pull-request-comment@52423e01640425a022ef5fd42c6fb5f633a02728 # v2.9.3
5881
with:
5982
recreate: true
6083
path: code-coverage-results.md
61-
number: ${{ steps.get_pr_number.outputs.pr_number }}
84+
number: ${{ needs.get-pr-number.outputs.pr_number }}
6285
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

0 commit comments

Comments
 (0)