Skip to content

chore: make zizmor configuration less verbose - #595

Merged
bepri merged 2 commits into
mainfrom
work/configure-zizmor
Aug 20, 2026
Merged

chore: make zizmor configuration less verbose#595
bepri merged 2 commits into
mainfrom
work/configure-zizmor

Conversation

@lengau

@lengau lengau commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

This configures zizmor just once to allow any starflow workflows without hash pinning.


  • I've followed the contribution guidelines.
  • I've signed the CLA.
  • I've successfully run make lint && make test.
  • I've added or updated any relevant documentation.
  • In documents I changed, I added a meta description if one was missing.
  • I've updated the relevant release notes.

Copilot AI lite review requested due to automatic review settings August 20, 2026 14:43
@lengau
lengau requested a review from bepri as a code owner August 20, 2026 14:43
@lengau
lengau requested a review from mr-cal August 20, 2026 14:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR centralizes zizmor configuration to exempt reusable workflows from canonical/starflow from SHA pinning requirements, allowing @main references without repeating inline zizmor: ignore[...] annotations across workflows.

Changes:

  • Added a repository zizmor config to allow ref-pinning for canonical/starflow workflows.
  • Removed per-line zizmor: ignore[unpinned-uses] annotations from existing reusable-workflow uses: entries.
  • Simplified CI workflow YAML by reducing inline lint configuration noise.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/zizmor.yaml Adds centralized unpinned-uses policy configuration for canonical/starflow.
.github/workflows/tics.yaml Removes inline zizmor ignore from the Starflow reusable workflow reference.
.github/workflows/qa.yaml Removes inline zizmor ignores from Starflow lint/test reusable workflow references.
.github/workflows/policy.yaml Removes inline zizmor ignores from Starflow policy and scan reusable workflow references.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/zizmor.yaml

@mr-cal mr-cal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Smart!

@mr-cal
mr-cal requested a review from a team August 20, 2026 16:16
@bepri
bepri merged commit 2e2d6c9 into main Aug 20, 2026
17 checks passed
@bepri
bepri deleted the work/configure-zizmor branch August 20, 2026 16:45
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.

4 participants