Skip to content

Commit 8931442

Browse files
authored
Merge pull request #220 from lukaszstolarczuk/codeql-permissions
Don't use CodeQL in reusable workflow
2 parents 84559ef + 43c8263 commit 8931442

File tree

2 files changed

+5
-7
lines changed

2 files changed

+5
-7
lines changed

.github/workflows/codeql.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
name: CodeQL
22

3-
on: workflow_call
3+
# Due to lower score on Scorecard we're running this separately from
4+
# "PR/push" workflow. For some reason permissions weren't properly set
5+
# or recognized (by Scorecard). If Scorecard changes its behavior we can
6+
# go back to use 'workflow_call' trigger.
7+
on: [push, pull_request]
48

59
permissions:
610
contents: read

.github/workflows/pr_push.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -85,9 +85,3 @@ jobs:
8585
Benchmark:
8686
needs: [Build]
8787
uses: ./.github/workflows/benchmarks.yml
88-
CodeQL:
89-
permissions:
90-
contents: read
91-
security-events: write
92-
needs: [Build]
93-
uses: ./.github/workflows/codeql.yml

0 commit comments

Comments
 (0)