Skip to content

Commit

Permalink
Move permissions to top level job definition (#107)
Browse files Browse the repository at this point in the history
It seems that we in the previous commit missed that permissions need to
be set either on all jobs or globally if all jobs should have the same
ones. This led to faulty/no permissions when trying to build and push
the goer image.

Change-Id: I3acbed8b05c9951c2a1658fd2cdf3a979b8fc20a

Co-authored-by: Eric Stengard <[email protected]>
  • Loading branch information
ericstengard and Eric Stengard authored Jan 29, 2024
1 parent 4a85d32 commit bf3a272
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/build-test-push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,14 @@ env:
IMAGE_NAME: ${{ github.repository }}

jobs:

permissions:
contents: read
packages: write

run-tests:
runs-on: ubuntu-latest

permissions:
contents: read
packages: write

steps:
- uses: actions/checkout@v4
with:
Expand Down

0 comments on commit bf3a272

Please sign in to comment.