Skip to content

ci: adopt the template's zizmor policy comment - #782

Merged
CybotTM merged 1 commit into
mainfrom
fix/sync-zizmor-template
Aug 2, 2026
Merged

ci: adopt the template's zizmor policy comment#782
CybotTM merged 1 commit into
mainfrom
fix/sync-zizmor-template

Conversation

@CybotTM

@CybotTM CybotTM commented Aug 2, 2026

Copy link
Copy Markdown
Member

main has been failing the template-drift check since 13:47Z today, and every open PR inherits it.

The go-app template reworded one comment in netresearch/.github@00f5783 — from "reusable workflows" to "uses: — reusable workflows AND composite actions" — and this consumer was never synced. Only the comment differs; "netresearch/*": ref-pin and "*": hash-pin are byte-identical either side.

The file is now byte-identical to templates/go-app/.github/zizmor.yml (sha256 bcf5a56f…), which is what the drift job compares against.

Worth noting for the fleet: every other repo on the go-app template that has not adopted this comment is red for the same reason. This PR only fixes ofelia.

The go-app template reworded this comment in netresearch/.github@00f5783
to cover composite actions as well as reusable workflows. The consumer
was not synced, so the template-drift check has failed on main since
that landed. Only the comment differs; the policies are unchanged.

Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
Copilot AI review requested due to automatic review settings August 2, 2026 15:21
@sonarqubecloud

sonarqubecloud Bot commented Aug 2, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Aligns this repo’s Zizmor policy comment with the upstream go-app template to resolve template-drift CI failures.

Changes:

  • Updates the explanatory comment under unpinned-uses.policies to match the template wording.
  • Clarifies that the first-party uses: policy applies to both reusable workflows and composite actions.

@codecov

codecov Bot commented Aug 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.28%. Comparing base (df222a9) to head (ab6184e).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #782   +/-   ##
=======================================
  Coverage   89.28%   89.28%           
=======================================
  Files          88       88           
  Lines       12147    12147           
=======================================
  Hits        10846    10846           
  Misses       1001     1001           
  Partials      300      300           
Flag Coverage Δ
integration 89.27% <ø> (-0.02%) ⬇️
unittests 88.70% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@github-actions github-actions Bot added the ci label Aug 2, 2026
@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Automated approval for maintainer PR

All automated quality gates passed. See SECURITY_CONTROLS.md for compensating controls.

@CybotTM
CybotTM added this pull request to the merge queue Aug 2, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Aug 2, 2026
@CybotTM
CybotTM added this pull request to the merge queue Aug 2, 2026
Merged via the queue into main with commit 3874340 Aug 2, 2026
37 checks passed
@CybotTM
CybotTM deleted the fix/sync-zizmor-template branch August 2, 2026 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants