Skip to content

Fix zizmor lint#2902

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

Fix zizmor lint#2902
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: this only changes a dev-only lint dependency and its rule configuration, with no impact on runtime code paths.

Overview
Bumps the dev dependency zizmor from 1.22.0 to 1.23.1.

Updates zizmor.yml to disable the superfluous-actions rule to prevent new lint failures in CI.

Written by Cursor Bugbot for commit 73683d2. 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.

dependabot-cooldown:
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 rule disable in config

Medium Severity

The PR description states it disables both superfluous-actions and secrets-outside-env to fix lint failures, but only superfluous-actions is actually added to zizmor.yml. The secrets-outside-env rule is missing entirely. Several workflows (e.g., dependabot-merge.yml, release.yml) pass secrets via with: parameters rather than env:, which means zizmor 1.23.1 will still flag those usages and lint will continue to fail.

Fix in Cursor Fix in Web

@adamtheturtle adamtheturtle merged commit c7344f3 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