ci: converge ci-workflows references on v0.22.2 - #123
Merged
Conversation
Moves every remaining melodic-software/ci-workflows reference this repository owns to v0.22.2 (5776760254f8b63cba44e896f51604cb755350d9), completing Phase 6b-ii convergence for melodic-software/.github. This repository has no synced .github/standards/runner-policy/policy.json and runs no Runner policy step, so every ci.yml composite and the link-check.yml reusable were moved after diffing their contract (action.yml for composites, on.workflow_call for the reusable) between 906ae7ef379ea4d2b8497f64475dce1d3d8715c4 (v0.22.0) and 5776760254f8b63cba44e896f51604cb755350d9 (v0.22.2). No diffs were found for any of the ten distinct paths (markdown, typos, editorconfig, gitleaks, lychee-offline, actionlint, check-jsonschema, shellcheck, eol-renormalize, pr-contract composites, and the link-check.yml reusable), so no caller inputs required adjustment. Moved (in .github/workflows/ci.yml, 14 pin sites across 10 composite paths, and .github/workflows/link-check.yml, 1 reusable pin): - .github/actions/markdown - .github/actions/typos - .github/actions/editorconfig - .github/actions/gitleaks - .github/actions/lychee-offline - .github/actions/actionlint - .github/actions/check-jsonschema (5 call sites) - .github/actions/shellcheck - .github/actions/eol-renormalize - .github/actions/pr-contract - .github/workflows/link-check.yml Left untouched, both already at v0.22.2 before this change: - .github/actions/ci-status caller in ci.yml (re-pinned in #120) - .github/actions/managed-files-guard caller in .github/workflows/managed-files-guard.yml (SYNC-MANAGED file; the standards sync owns this pin, not this PR) Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019APkoJh8NVN773d82fBBSG
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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.
No related issue: melodic-software/github-iac#378 tracks the ci-perf program (Phase 6b-ii convergence)
Summary
Moves every remaining
melodic-software/ci-workflows/...reference this repository owns to v0.22.2 (5776760254f8b63cba44e896f51604cb755350d9), completing Phase 6b-ii convergence formelodic-software/.github.This repository has no synced
.github/standards/runner-policy/policy.jsonand runs noRunner policystep, so per the convergence rule every reusable was moved after diffing itson.workflow_callcontract (and every composite after diffing itsaction.yml) between the old SHA and the new one, rather than gating on a policy-approved contract.Fix
Diffed every distinct path's contract between
906ae7ef379ea4d2b8497f64475dce1d3d8715c4(v0.22.0) and5776760254f8b63cba44e896f51604cb755350d9(v0.22.2) viagh api "repos/melodic-software/ci-workflows/contents/<path>?ref=<sha>" --jq .content | base64 -d. No diffs were found for any of the ten paths, so no caller inputs, secrets, or permissions required adjustment.Moved (in
.github/workflows/ci.yml, 14 pin sites across 10 composite paths, and.github/workflows/link-check.yml, 1 reusable pin):.github/actions/markdown.github/actions/typos.github/actions/editorconfig.github/actions/gitleaks.github/actions/lychee-offline.github/actions/actionlint.github/actions/check-jsonschema(5 call sites).github/actions/shellcheck.github/actions/eol-renormalize.github/actions/pr-contract.github/workflows/link-check.ymlLeft untouched, both already at v0.22.2 before this change:
.github/actions/ci-statuscaller inci.yml(re-pinned in ci: re-pin ci-status to ci-workflows v0.22.2 #120).github/actions/managed-files-guardcaller in.github/workflows/managed-files-guard.yml(SYNC-MANAGED file; the standards sync owns this pin, not this PR)Verification
All 17 references in this repository are now at v0.22.2; there are no other SHAs to explain.
actionlint -shellcheck= -pyflakes=: clean, no findings.Runner policystep exists in this repository (no syncedpolicy.json); not applicable..githubin the convergence brief.Related
Refs melodic-software/github-iac#378