Skip to content

Fix zizmor warnings - #159

Merged
biow0lf merged 1 commit into
masterfrom
fix-zizmor-warnings
Oct 2, 2026
Merged

biow0lf merged 1 commit into
masterfrom
fix-zizmor-warnings

Conversation

@biow0lf

@biow0lf biow0lf commented Oct 2, 2026

Copy link
Copy Markdown
Member

No description provided.

@biow0lf biow0lf self-assigned this Oct 2, 2026
@biow0lf
biow0lf marked this pull request as ready for review October 2, 2026 13:27
Copilot AI balanced review requested due to automatic review settings October 2, 2026 13:27

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

🟢 Approval recommended

The narrowly scoped security changes have no reported blocking issues or downstream dependency on persisted credentials.

Review effort: Balanced
Findings: None

What changed in this PR

Addresses zizmor warnings by disabling persisted checkout credentials in the documentation workflows.

Changes:

  • Sets persist-credentials: false for both Docker workflow checkouts.
  • Applies the same setting to the dependency-review checkout.
File Description
.github/​workflows/​docker.yml Disables credential persistence for both repository checkouts.
.github/​workflows/​dependency-review.yml Disables credential persistence for the dependency-review checkout.

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

@biow0lf
biow0lf merged commit 138994b into master Oct 2, 2026
7 checks passed
@biow0lf
biow0lf deleted the fix-zizmor-warnings branch October 2, 2026 13:30
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