Skip to content

Commit 518ea14

Browse files
Fix testing permissions (#2517)
1 parent 697430b commit 518ea14

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

.github/workflows/run_tests.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,10 @@ on:
2222
pull_request_review:
2323
types: [submitted]
2424

25+
permissions:
26+
contents: read
27+
pull-requests: write
28+
2529
jobs:
2630
determine-test-scope:
2731
runs-on: [ deploynode ]
@@ -185,12 +189,8 @@ jobs:
185189
echo "total=$TOTAL_COVERAGE" >> "$GITHUB_ENV"
186190
echo "### Total coverage: ${TOTAL_COVERAGE}%"
187191
188-
- name: verify-coverage-report-location
189-
run: |
190-
ls $GITHUB_WORKSPACE
191-
192192
- name: Produce the diff coverage report
193-
if: matrix.python-version == '3.13'
193+
if: matrix.python-version == '3.13' && github.event_name == 'pull_request'
194194
run: |
195195
source ${GITHUB_WORKSPACE}/.venv/bin/activate
196196
git config --global --add safe.directory ${GITHUB_WORKSPACE}
@@ -199,7 +199,7 @@ jobs:
199199
--format markdown:diff-coverage.md
200200
201201
- uses: actions/github-script@v7
202-
if: matrix.python-version == '3.13'
202+
if: matrix.python-version == '3.13' && github.event_name == 'pull_request'
203203
with:
204204
result-encoding: string
205205
script: |

0 commit comments

Comments
 (0)