Skip to content

Zizmor fixes - #47

Merged
bb-froggy merged 4 commits into
masterfrom
zizmor-fixes
Sep 28, 2026
Merged

bb-froggy merged 4 commits into
masterfrom
zizmor-fixes

Conversation

@bb-froggy

Copy link
Copy Markdown
Member

No description provided.

@bb-froggy
bb-froggy requested a lite review from Copilot September 28, 2026 09:23
@github-actions

github-actions Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

🌈 zizmor scan results

✅ zizmor found no issues in this repository's GitHub Actions.

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.

Copilot review overview

🟡 Changes recommended

Critical invalid reusable-workflow paths must be corrected before approval.

Review effort: Lite
Findings: 2 High severity

Open (2)
What changed in this PR

Updates GitHub Actions workflows to address Zizmor recommendations through pinned actions, least-privilege permissions, and hardened checkout settings.

Changes:

  • Pins third-party actions to commit SHAs.
  • Restricts workflow permissions.
  • Disables unnecessary checkout credentials.
File Summary
.github/​workflows/​prepare-marketplace.yml Hardens permissions, checkout, and artifact actions.
.github/​workflows/​generate-arm-template.yml Restricts permissions and checkout credentials; critical invalid reusable-workflow path requires correction.
.github/​workflows/​dependabot-auto-merge-actions.yml Pins the Dependabot metadata action.
.github/​workflows/​check-arm-template.yml Adds read-only permissions and secure checkout settings.
.github/​workflows/​analyzeStyle.yml Hardens checkout settings; critical invalid reusable-workflow path requires correction.

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

Comment thread .github/workflows/analyzeStyle.yml
Comment thread .github/workflows/generate-arm-template.yml
@bb-froggy
bb-froggy merged commit 484b44f into master Sep 28, 2026
7 checks passed
@bb-froggy
bb-froggy deleted the zizmor-fixes branch September 28, 2026 10:03
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