Skip to content

ci: fix zizmor audit findings#591

Merged
dsanders11 merged 3 commits intomainfrom
ci/zizmor-audit-fixes
Apr 2, 2026
Merged

ci: fix zizmor audit findings#591
dsanders11 merged 3 commits intomainfrom
ci/zizmor-audit-fixes

Conversation

@MarshallOfSound
Copy link
Copy Markdown
Member

Fixes zizmor audit findings surfaced in the latest scheduled audit run.

Changes:

  • dependabot.yml: add 7-day cooldown for github-actions updates (dependabot-cooldown).
  • release.yml: add top-level permissions: {} and minimal contents: read for the called test job (excessive-permissions).
  • test.yml: add persist-credentials: false to actions/checkout (artipacked).

Remaining (not auto-fixable):

  • add-to-project.yml uses pull_request_target (dangerous-triggers). The workflow does not check out untrusted PR code; it only generates an app token and adds the PR/issue to a project board. Switching to pull_request would break project triage for fork PRs since secrets are unavailable on the pull_request event from forks. Left as-is.

cc @dsanders11

@dsanders11 dsanders11 marked this pull request as ready for review April 2, 2026 21:07
@dsanders11 dsanders11 requested a review from a team as a code owner April 2, 2026 21:07
@dsanders11 dsanders11 merged commit 6cdc9f4 into main Apr 2, 2026
11 of 19 checks passed
@dsanders11 dsanders11 deleted the ci/zizmor-audit-fixes branch April 2, 2026 21:49
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