Skip to content

Commit 9c0d70e

Browse files
committed
Grant the notify job contents: read for its checkout
Both reviewers caught it: the workflow sets permissions: {} globally, and the checkout added for notify-issue.sh gets no repository access from issues: write alone — the job would die before filing the issue, exactly on the runs it exists for. Same non-inheritance note as release.yml documents for its headless-probe call.
1 parent 11b2276 commit 9c0d70e

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

‎.github/workflows/installer-smoke.yml‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,11 @@ jobs:
131131
if: failure()
132132
runs-on: ubuntu-latest
133133
timeout-minutes: 5
134+
# contents: read cannot be inherited — this workflow sets permissions: {}
135+
# at the top, so without it the checkout that fetches notify-issue.sh has
136+
# no repository access and the job dies before it can file the issue.
134137
permissions:
138+
contents: read
135139
issues: write
136140
steps:
137141
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1

0 commit comments

Comments
 (0)