chore: bump pinned versions - #55
Open
github-actions[bot] wants to merge 1 commit into
Open
Conversation
chaudharydeepanshu
marked this pull request as ready for review
September 1, 2026 11:32
|
i went through this but came up with nothing to post. ping me again if i missed something ✨ |
slopfairy
requested changes
Sep 1, 2026
slopfairy
left a comment
There was a problem hiding this comment.
Caution
Requesting changes — zizmor 1.30.0 fails the repo's own workflow-lint gate
🔴 Blocking — must fix before merge
tool/versions.env:36— bumpingZIZMOR_VERSION1.29.0 → 1.30.0 turns theworkflow-lintjob red. zizmor 1.30.0 ships a newself-repositoryaudit that flags threeuses: ./actions/capabilities/oci-cachesites in.github/workflows/self-check.yml(lines 172, 184, 204); with--persona=auditorthe job exits 12 (3 low findings). The PR body says merge once self-check is green, and this bump is exactly what makes it red. Fix in the same PR by applying zizmor's auto-fix at those three sites (uses: $/actions/capabilities/oci-cache), or drop the zizmor bump until self-check.yml is migrated.self-upgrade.ymlonly rewritesversions.env, so this needs a hand edit.
✅ Checked
- Bump is legitimately sourced:
upgrade.shpulls the version from PyPI's JSON, and the failing job log confirmspipx run "zizmor==1.30.0"resolved and installed from PyPI. - Only
ZIZMOR_VERSIONchanged; no sha256 moved. zizmor has no pinned hash by design (pipx resolves from PyPI), and every other pin (fvm, chrome, bore, oras, actionlint, pinact) is untouched. - Conventional Commit, Action YAML parse, Internal refs are @main, Pin availability, and all three oci-cache round-trips pass.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Supply-chain pin bump — review the recomputed hashes in the diff:
tool/versions.env: sha256srecomputed from the upstream release assets, re-verified by
fetch_verifiedon every build. See the diff for which moved.These pins are consumed by every whuppi/ci consumer. The already-pinned
assets were re-hashed before this bump (repoint alarm). Merge once
self-check is green; cut a whuppi/ci release so consumers pick the bump
up through their grouped Dependabot PR.
Auto-generated by self-upgrade.yml