Skip to content

Zizmor fixes - #6

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

bb-froggy merged 7 commits into
masterfrom
zizmor-fixes

Conversation

@bb-froggy

Copy link
Copy Markdown
Member

Fixes for zizmor findings and auto-merge dependabot PRs if they affect only actions

@bb-froggy

Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request. Please use the latest checkout version, but use pinned commits.

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

The Dependabot auto-merge workflow cannot obtain required write permissions and may merge before checks complete.

Review effort: Lite
Findings: 1 High severity

Open (1)
What changed in this PR

This PR hardens GitHub Actions workflows and adds automatic merging for Dependabot Actions updates.

Changes:

  • Pins third-party actions and restricts permissions.
  • Improves workflow input handling.
  • Adds Dependabot cooldown configuration.
  • Adds Dependabot Actions auto-merge automation.
File Summary
.github/​workflows/​dependabot-auto-merge-actions.yml Adds Dependabot approval and merge automation; requires fixes for token permissions and merge timing.
.github/​workflows/​auto-publish-in-azure.yml Hardens permissions, action references, and shell inputs.
.github/​dependabot.yml Adds a seven-day dependency cooldown.

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

# Other Dependabot PRs (npm, nuget, …) are left untouched.

name: Dependabot auto-merge GitHub Actions
on: pull_request

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That's incorrect

# Conflicts:
#	.github/workflows/auto-publish-in-azure.yml

Co-authored-by: bb-froggy <587120+bb-froggy@users.noreply.github.com>

Copilot AI commented Sep 28, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request. Please use the latest checkout version, but use pinned commits.

Done in 1c602b4. I merged the latest master into this branch and resolved the checkout conflicts by pinning both workflow uses to actions/checkout v7.0.1 (3d3c42e5aac5ba805825da76410c181273ba90b1).

@bb-froggy
bb-froggy merged commit 85ba00f into master Sep 28, 2026
2 checks passed
@github-actions

Copy link
Copy Markdown

🌈 zizmor scan results

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

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.

3 participants