Skip to content

Commit 86f82a1

Browse files
authored
Apply suggestions from code review
Co-authored-by: Jan Kowalleck <jan.kowalleck@gmail.com> Signed-off-by: Jan Kowalleck <jan.kowalleck@gmail.com>
1 parent f00fafd commit 86f82a1

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

.github/workflows/zizmor.yml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ on:
77
pull_request:
88
paths:
99
- '.github/workflows/**'
10+
push:
11+
paths:
12+
- ".github/workflows/**"
1013
schedule:
1114
# Every Saturday 00:00 UTC
1215
- cron: '0 0 * * 6'
@@ -21,7 +24,7 @@ jobs:
2124
zizmor:
2225
name: zizmor
2326
runs-on: ubuntu-latest
24-
timeout-minutes: 5
27+
timeout-minutes: 10
2528
permissions:
2629
contents: read
2730
steps:
@@ -34,5 +37,11 @@ jobs:
3437
# see https://github.com/zizmorcore/zizmor-action
3538
uses: zizmorcore/zizmor-action@b1d7e1fb5de872772f31590499237e7cce841e8e # v0.5.3
3639
with:
40+
# advanced-security: false => emit findings as workflow-command annotations (::error file=…) rather than
41+
# uploading a SARIF report to GitHub's Security tab.
42+
# Uploading SARIF requires `security-events: write` and GitHub Advanced Security (GHAS),
43+
# both of which are unnecessary here and would violate the least-privilege policy.
44+
# The two modes are mutually exclusive: advanced-security must be false for
45+
# annotations to take effect.
3746
advanced-security: false
3847
annotations: true

0 commit comments

Comments
 (0)