Skip to content

Commit e697914

Browse files
Merge pull request #121 from HendrikBorgelt/fix-excel-inbox-fork-checkout
Fix excel_inbox.yaml failing on every fork PR that touches inbox/
2 parents b04e4c5 + 799d152 commit e697914

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

.github/workflows/excel_inbox.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,11 @@ jobs:
7979
ref: ${{ github.event.pull_request.head.ref }}
8080
fetch-depth: 1
8181
persist-credentials: false
82+
# actions/checkout@v7 refuses fork checkouts under pull_request_target
83+
# by default (anti "pwn request" guard). Safe to opt in here: this
84+
# checkout is read-only data (the xlsx is never executed) and every
85+
# script that actually runs is checked out from main below.
86+
allow-unsafe-pr-checkout: true
8287

8388
- name: Check out main branch into _main_branch/
8489
# Always run OUR scripts from main — never from the PR.
@@ -268,6 +273,8 @@ jobs:
268273
ref: ${{ github.event.pull_request.head.ref }}
269274
fetch-depth: 0
270275
persist-credentials: true
276+
# See the apply-inbox job for why this opt-in is safe here.
277+
allow-unsafe-pr-checkout: true
271278

272279
- name: Check out main branch into _main_branch/
273280
uses: actions/checkout@v7.0.0

0 commit comments

Comments
 (0)