← Back to issue list

ci: remove deprecated flag for zimzor

View original Github issue

Metadata

Project
starflow
Number
#167
Type
pull request
State
merged
Author
bepri
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR canonical/starflow#167 by maintainer bepri drops zizmor's deprecated `--upload-sarif` flag from the CI workflow (1 file, -2 lines) since SARIF upload is now the default, clearing recurring warnings.

Resolution: closed resolved

Reason: The pull request was approved by one reviewer and merged into the default branch, deleting the two deprecated-flag lines from the zizmor GitHub Actions workflow. All 28 CI checks, including zizmor, GitHub Actions security audit, and the OSV-scanner jobs, passed with no failures.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The `upload-sarif` flag was deprecated and putting warnings in all of our CI runs. It seems to always upload a SARIF by default.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR canonical/starflow#167 by maintainer bepri drops zizmor's deprecated `--upload-sarif` flag from the CI workflow (1 file, -2 lines) since SARIF upload is now the default, clearing recurring warnings.
stealth/space-bunny-alpha — closed resolved Merged canonical/starflow#167, removing zizmor’s deprecated --upload-sarif flag because SARIF uploads now occur by default, eliminating recurring CI warnings.
qwen/qwen3.6-35b-a3b — — Merged removal of the deprecated upload-sarif flag from zizmor CI configuration. This resolves recurring CI warnings by relying on default SARIF upload behavior. Approved by one reviewer with all checks passing.
qwen/qwen3.6-35b-a3b
Staleness: 0
Complexity: 0
Confidence: 95
needs review Removes deprecated upload-sarif flag from CI config to eliminate warnings, as SARIF upload is now default. Approved by one reviewer, all CI checks passing, ready to merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 5
Confidence: 85
needs review Removes deprecated upload-sarif flag from Zizmor CI config to eliminate warnings, as SARIF upload is now default. All CI checks passing. Awaiting maintainer review.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 5
Confidence: 90
needs review Removes deprecated upload-sarif flag from zizmor CI configuration to eliminate warnings, as SARIF upload is now default. Currently pending maintainer review with most CI checks passing and a few slow tests pending.

Update history

Date Change
updated
updated
updated
created

Related issues

Issue Project State Summary Similarity
#6393 ci: fix zizmor warnings snapcraft merged Merged PR #6393 as commit c41c04835, correcting Zizmor warnings originating from the Starbase merge in #6391.
71%