Skip to content

Commit

Permalink
fix: ci token permissions for pr push
Browse files Browse the repository at this point in the history
- fix: ci token permissions for pr push
- fix: concurrency for ci push jobs

Fixes and closes #114
https://github.com/sgammon/rules_graalvm/security/code-scanning/42

Signed-off-by: Sam Gammon <sam@elide.ventures>
  • Loading branch information
sgammon committed Sep 8, 2023
1 parent c4c02a1 commit 1fa60fe
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/on.pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,9 @@ concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
cancel-in-progress: true

permissions:
contents: read

jobs:
dependency-graph:
name: "Dependency Graph"
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/on.push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,11 @@ name: "CI"
- "*.bzl"
- "*.bazel"

concurrency:
# Cancel previous actions from the same PR: https://stackoverflow.com/a/72408109
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true

jobs:
dependency-graph:
name: "Dependency Graph"
Expand Down

0 comments on commit 1fa60fe

Please sign in to comment.