Skip to content

Fix zizmor lint#1584

Merged
adamtheturtle merged 2 commits intomainfrom
fix-zizmor-lint
Mar 10, 2026
Merged

Fix zizmor lint#1584
adamtheturtle merged 2 commits intomainfrom
fix-zizmor-lint

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Mar 10, 2026

Upgrade zizmor to 1.23.1 and disable superfluous-actions and secrets-outside-env to fix lint failures.

Made with Cursor


Note

Low Risk
Low risk config-only change that relaxes a zizmor lint rule; it does not affect runtime behavior.

Overview
Disables zizmor's superfluous-actions rule in zizmor.yml to prevent lint failures.

Written by Cursor Bugbot for commit 1a744c7. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

template-injection:
disable: true
superfluous-actions:
disable: true
Copy link

Choose a reason for hiding this comment

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

Missing secrets-outside-env disable rule in zizmor config

Medium Severity

The PR description states both superfluous-actions and secrets-outside-env rules need to be disabled to fix lint failures, but only superfluous-actions was added to zizmor.yml. The workflow files (e.g., release.yml, dependabot-merge.yml) pass secrets directly in action with: parameters rather than through env: blocks, so the secrets-outside-env rule will still cause zizmor lint failures. This means the PR's stated goal of fixing lint is only partially achieved.

Fix in Cursor Fix in Web

@adamtheturtle adamtheturtle merged commit a22d5aa into main Mar 10, 2026
16 checks passed
@adamtheturtle adamtheturtle deleted the fix-zizmor-lint branch March 10, 2026 10:16
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.

1 participant