ci: adds zizmor action - #144
bmuenzenmeyer wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/web-infra Please review the changes when you have a chance. Thank you! 🙏 |
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Fix the invalid branch filter and prevent fork pull request SARIF uploads from failing.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 1
Open (1)
What changed in this PR
Adds a GitHub Actions workflow to scan repository workflows with zizmor.
Changes:
- Runs on pushes and pull requests targeting
main. - Uses pinned actions and restricted permissions.
- Uploads SARIF results to GitHub Advanced Security.
| File | Review findings |
|---|---|
.github/workflows/zizmor.yml |
Critical: branches must use list syntax; the current scalar form invalidates the workflow. Moderate: Advanced Security uploads can fail for fork pull requests due to restricted token permissions. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Matt Cowley <me@mattcowley.co.uk> Signed-off-by: Claudio Wunder <cwunder@gnome.org>

Same as doc-kit