feat: add reusable zizmor scan workflow for consumer repos - #81
Open
michaelpeterswa wants to merge 1 commit into
Open
feat: add reusable zizmor scan workflow for consumer repos#81michaelpeterswa wants to merge 1 commit into
michaelpeterswa wants to merge 1 commit into
Conversation
Publishes zizmor_scan.yml as a workflow_call reusable workflow so any repo in the org can add zizmor scanning to its own CI with a single `uses:` line, rather than everyone hand-rolling the setup we built in #78/#79. Defaults to audit-only (fail-on-findings: false) to match the rollout approach used in this repo: annotate and comment, don't block, so adopting repos aren't immediately red on day one. Reports via GitHub annotations rather than SARIF, since most consumer repos are private and don't have a GitHub Advanced Security license (see prior discussion in #76). Exposes fail-on-findings, comment, persona, and min-severity as inputs so each repo can tune it once they're ready. Uses its own zizmor/{version} tag prefix in the README's versioning scheme, since it isn't tied to a language stack the way go_app/php_lib etc. are. Part of #76. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
🌈 zizmor findingsFound 96 finding(s) across the workflows in this PR.
See the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Publishes zizmor_scan.yml as a workflow_call reusable workflow so any
repo in the org can add zizmor scanning to its own CI with a single
uses:line, rather than everyone hand-rolling the setup we built in#78/#79.
Defaults to audit-only (fail-on-findings: false) to match the rollout
approach used in this repo: annotate and comment, don't block, so
adopting repos aren't immediately red on day one. Reports via GitHub
annotations rather than SARIF, since most consumer repos are private
and don't have a GitHub Advanced Security license (see prior
discussion in #76). Exposes fail-on-findings, comment, persona, and
min-severity as inputs so each repo can tune it once they're ready.
Uses its own zizmor/{version} tag prefix in the README's versioning
scheme, since it isn't tied to a language stack the way go_app/php_lib
etc. are.
Test plan
zizmoron this file (both regular and pedantic personas): 0 findings.actionlint: clean.zizmorandPR summary commentchecks (inherited from Add zizmor static analysis gate for GitHub Actions workflows #78) both pass on this PR.zizmor/v1andzizmor/v1.0.0tags per this repo's versioning convention, since consumer repos need a tag to pin to (not@main).uses:call works cross-repo before announcing broadly.Part of #76.
Stack created with GitHub Stacks CLI • Give Feedback 💬