Skip to content

Commit 5dac70d

Browse files
committed
Remove code coverage PR comment. Refine ci permissions.
1 parent 2f404a5 commit 5dac70d

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

.github/workflows/ci.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@ on:
66
pull_request:
77
branches: ["main"]
88

9+
permissions:
10+
contents: read
11+
912
jobs:
1013
build:
11-
permissions:
12-
contents: read
13-
1414
strategy:
1515
matrix:
1616
os: [ubuntu-latest, windows-latest, macos-latest]

.github/workflows/code-coverage.yml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@ name: Code Coverage
33
on:
44
workflow_call:
55

6+
permissions:
7+
contents: read
8+
69
jobs:
710
publish-coverage:
811
runs-on: ubuntu-latest
@@ -59,13 +62,6 @@ jobs:
5962
path: ${{ github.workspace }}/code-coverage-results.md
6063
retention-days: 7
6164

62-
- name: Add Coverage PR Comment
63-
uses: marocchino/sticky-pull-request-comment@v2
64-
if: github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository
65-
with:
66-
recreate: true
67-
path: ${{ github.workspace }}/code-coverage-results.md
68-
6965
- name: Coverage on step summary
7066
if: always()
7167
run: cat "${{ github.workspace }}/report/SummaryGithub.md" >> $GITHUB_STEP_SUMMARY

0 commit comments

Comments
 (0)