Skip to content

Commit 1c98f3e

Browse files
Bordamseeger
authored andcommitted
limit PR permissions vol.2 (Lightning-AI#2078)
1 parent 6eccf59 commit 1c98f3e

File tree

1 file changed

+7
-17
lines changed

1 file changed

+7
-17
lines changed

.github/workflows/cpu-tests.yml

Lines changed: 7 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -11,17 +11,12 @@ on:
1111

1212
# lock down all permissions by default
1313
permissions:
14-
contents: read
15-
issues: read
16-
pull-requests: read
17-
id-token: read
18-
security-events: read
19-
actions: read
20-
checks: write
21-
deployments: read
22-
discussions: read
23-
packages: read
24-
statuses: write
14+
contents: read # needed to check out code
15+
checks: write # needed for test results
16+
pull-requests: read # needed for PR metadata
17+
actions: read # needed to use actions
18+
security-events: none
19+
statuses: write # needed to update commit status
2520

2621
concurrency:
2722
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.head_ref }}
@@ -40,6 +35,7 @@ env:
4035
jobs:
4136
testing-imports:
4237
runs-on: ${{ matrix.os }}
38+
if: github.event_name != 'pull_request_target'
4339
strategy:
4440
fail-fast: false
4541
matrix:
@@ -49,12 +45,6 @@ jobs:
4945
steps:
5046
- name: Checkout generic
5147
uses: actions/checkout@v4
52-
if: github.event_name != 'pull_request_target'
53-
- name: Checkout for `pull_request_target`
54-
uses: actions/checkout@v4
55-
if: github.event_name == 'pull_request_target'
56-
with:
57-
ref: ${{ github.event.pull_request.head.sha }}
5848
- uses: actions/setup-python@v5
5949
with:
6050
python-version: ${{ matrix.python-version }}

0 commit comments

Comments
 (0)