Skip to content

Commit 111599c

Browse files
ci: colocate zizmor suppression rationales
1 parent c681a78 commit 111599c

7 files changed

Lines changed: 21 additions & 16 deletions

.github/workflows/codeql.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,9 @@ jobs:
5858
# and so it uses more parallelism which uses more memory
5959
sed -i "s/org.gradle.jvmargs=/org.gradle.jvmargs=-Xmx8g /" gradle.properties
6060
61-
# zizmor cache-poisoning suppression rationale: this workflow analyzes release branches but does not build or publish release artifacts.
62-
- name: Set up Gradle # zizmor: ignore[cache-poisoning]
61+
- name: Set up Gradle
62+
# zizmor: ignore[cache-poisoning] This workflow analyzes release branches
63+
# but does not build or publish release artifacts.
6364
if: matrix.language == 'java'
6465
uses: gradle/actions/setup-gradle@3f131e8634966bd73d06cc69884922b02e6faf92 # v6.2.0
6566
with:

.github/workflows/comment-on-test-label.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,13 @@ name: Comment on test-* label
1212
# `pull_request_target` would normally expose are not reachable.
1313

1414
on:
15-
pull_request_target: # zizmor: ignore[dangerous-triggers]
15+
pull_request_target:
16+
# zizmor: ignore[dangerous-triggers] No PR code is checked out or executed;
17+
# the workflow only posts a comment using trusted workflow code.
1618
types: [labeled]
1719

1820
permissions:
19-
pull-requests: write # zizmor: ignore[excessive-permissions]
21+
pull-requests: write # zizmor: ignore[excessive-permissions] Required to post the explanatory PR comment.
2022

2123
jobs:
2224
comment:

.github/workflows/pr-automation-comments.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
name: PR Automation Comments
22
on:
3-
# zizmor dangerous-triggers suppression rationale: required to comment on labeled fork PRs;
4-
# no PR code is checked out or executed.
5-
pull_request_target: # zizmor: ignore[dangerous-triggers]
3+
pull_request_target:
4+
# zizmor: ignore[dangerous-triggers] Required to comment on labeled fork PRs;
5+
# no PR code is checked out or executed.
66
types: [labeled]
77

88
permissions:
9-
pull-requests: write # zizmor: ignore[excessive-permissions]
9+
pull-requests: write # zizmor: ignore[excessive-permissions] Required to inspect and create PR comments.
1010

1111
jobs:
1212
comment-on-breaking-change:

.github/workflows/pr-label-automation.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ on:
44
types: [created]
55

66
permissions:
7-
pull-requests: write # zizmor: ignore[excessive-permissions]
7+
pull-requests: write # zizmor: ignore[excessive-permissions] Required to add the requested PR label.
88

99
jobs:
1010
auto-label:

.github/workflows/rerun-failed-daily-jobs.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
name: Rerun failed daily jobs
22

33
on:
4-
# zizmor dangerous-triggers suppression rationale: uses only the trusted default-branch rerun script
5-
# and treats the run ID as data.
6-
workflow_run: # zizmor: ignore[dangerous-triggers]
4+
workflow_run:
5+
# zizmor: ignore[dangerous-triggers] Uses only the trusted default-branch rerun
6+
# script and treats the run ID as data.
77
workflows:
88
- "Build (daily)"
99
- "Build (daily --no-build-cache)"

.github/workflows/rerun-flaky-pr-jobs.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
name: Rerun flaky PR jobs
22

33
on:
4-
# zizmor dangerous-triggers suppression rationale: uses only the trusted default-branch rerun script
5-
# and treats the run ID as data.
6-
workflow_run: # zizmor: ignore[dangerous-triggers]
4+
workflow_run:
5+
# zizmor: ignore[dangerous-triggers] Uses only the trusted default-branch rerun
6+
# script and treats the run ID as data.
77
workflows:
88
- "Build pull request"
99
- "CodeQL"

.github/workflows/survey-on-merged-pr.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
name: Survey on Merged PR by Non-Member
22

33
on:
4-
pull_request_target: # zizmor: ignore[dangerous-triggers] — this workflow only calls the reusable shared workflow; no PR code is checked out or executed.
4+
pull_request_target:
5+
# zizmor: ignore[dangerous-triggers] This only calls the trusted reusable
6+
# workflow; no PR code is checked out or executed.
57
types: [closed]
68
branches: [main]
79

0 commit comments

Comments
 (0)