ci: converge ci-workflows references on v0.22.2 - #45
Merged
Conversation
Moves the pr-contract composite reference in .github/workflows/ci.yml from melodic-software/ci-workflows@906ae7e # v0.22.0 to @5776760254f8b63cba44e896f51604cb755350d9 # v0.22.2. Diffed action.yml between the two SHAs; the composite's inputs, outputs, and run step are byte-identical, so no caller adjustment was needed. ci-status was already re-pinned to v0.22.2 by #42. managed-files-guard.yml carries the managed-files-guard composite at v0.22.2 already, but is SYNC-MANAGED (header present) and is left untouched here regardless. This repository has no .github/actions directory (no locally defined composites) and no synced .github/standards/runner-policy/policy.json, so per the brief every reusable/composite reference is moved after a diff rather than gated by an approved-contract list. 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. |
kyle-sexton
changed the title
kyle-sexton
changed the title
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
1 participant
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
Converges the last remaining
melodic-software/ci-workflowsreference in this repository on v0.22.2 (5776760254f8b63cba44e896f51604cb755350d9).Fix
.github/workflows/ci.yml,ci-statusjob,pr-contractcomposite:@906ae7ef379ea4d2b8497f64475dce1d3d8715c4 # v0.22.0to@5776760254f8b63cba44e896f51604cb755350d9 # v0.22.2. Diffedaction.ymlbetween the two SHAs; inputs, outputs, and the run step are byte-identical, so no caller adjustment was needed..github/workflows/ci.yml,ci-statusjob,ci-statuscomposite, already@5776760254f8b63cba44e896f51604cb755350d9 # v0.22.2from ci: re-pin ci-status to ci-workflows v0.22.2 #42..github/workflows/managed-files-guard.yml,managed-files-guardcomposite, already@5776760254f8b63cba44e896f51604cb755350d9 # v0.22.2; the file carries aSYNC-MANAGEDheader and is standards-managed regardless..github/actionsdirectory (no locally defined composites) and no synced.github/standards/runner-policy/policy.json/Runner policystep, confirmed by inspection of the worktree. Everyci-workflowsreference is therefore moved after anaction.yml/on.workflow_calldiff rather than gated by an approved-contract list; there was exactly one reference left to move.Verification
All three
ci-workflowsreferences (pr-contract,ci-status,managed-files-guard) now pin5776760254f8b63cba44e896f51604cb755350d9 # v0.22.2; none remain unexplained.actionlint -shellcheck= -pyflakes=: clean.npm run validate(marketplace validator):PASS: marketplace ...; 0 error(s), 0 warning(s).npm test: 113/114 pass. The one failure,collector orders tied skill mentions deterministicallyintests/codex-operations.test.mjs, is a pre-existing Windows CRLF line-splitting artifact unrelated to this change (the diff touches only.github/workflows/ci.yml); confirmed withgit diff --stat.Related
Refs melodic-software/github-iac#378