Skip to content

Commit

Permalink
ci: set permissions for contents and pull_requests to write
Browse files Browse the repository at this point in the history
Permission of pull_requests is required for leaving a comment on alert
on PR.

Permission of contents is required to be `write` when pushing to
gh-pages, originally set to `read` still worked might due to the write
permission had already been set somewhere else (repo/org-wise).

Signed-off-by: Thing-han, Lim <[email protected]>
  • Loading branch information
potsrevennil authored and mkannwischer committed Nov 21, 2024
1 parent 94c85ef commit 400ee2b
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions .github/workflows/bench.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
# SPDX-License-Identifier: Apache-2.0

name: Bench
permissions:
contents: read
on:
workflow_dispatch:
push:
Expand All @@ -17,6 +15,9 @@ concurrency:

jobs:
bench:
permissions:
contents: write
pull-requests: write
name: ${{ matrix.target.name }}
strategy:
fail-fast: true
Expand Down Expand Up @@ -56,8 +57,9 @@ jobs:
ec2_all:
name: ${{ matrix.target.name }} ${{ matrix.opt.name }}
permissions:
contents: 'read'
id-token: 'write'
contents: write
pull-requests: write
id-token: write
strategy:
fail-fast: false
matrix:
Expand Down

0 comments on commit 400ee2b

Please sign in to comment.