Skip to content

fix(ci): add security-events:write for zizmor SARIF upload#62

Merged
iamladi merged 1 commit intomainfrom
fix/ci-zizmor-permissions
Apr 2, 2026
Merged

fix(ci): add security-events:write for zizmor SARIF upload#62
iamladi merged 1 commit intomainfrom
fix/ci-zizmor-permissions

Conversation

@iamladi
Copy link
Copy Markdown
Owner

@iamladi iamladi commented Apr 2, 2026

Summary

CI failed on main after merging #61 — the zizmor-action uploads SARIF to GitHub Code Scanning, which requires security-events: write permission. The workflow only had contents: read.

Test plan

  • CI passes — Lint GitHub Actions job completes successfully

The zizmor-action uploads findings to GitHub Code Scanning via SARIF,
which requires security-events:write permission. Without it the action
fails with "Resource not accessible by integration".

permissions:
contents: read
security-events: write
@iamladi iamladi merged commit 0f2d00a into main Apr 2, 2026
8 checks passed
@iamladi iamladi deleted the fix/ci-zizmor-permissions branch April 2, 2026 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants