-
-
Notifications
You must be signed in to change notification settings - Fork 0
Submodule pins drift to orphaned commits — re-pin done; harden against recurrence #15
Copy link
Copy link
Open
Labels
cicdCI/CD: workflows, actions, lockfiles, pins, runners, release gatesCI/CD: workflows, actions, lockfiles, pins, runners, release gatesmeta:campaignCoordinated multi-repo push with a defined end stateCoordinated multi-repo push with a defined end statepriority:p3Low - nice to haveLow - nice to havescope:estateAffects many or all repos across the estateAffects many or all repos across the estate
Description
Activity
Metadata
Metadata
Assignees
Labels
cicdCI/CD: workflows, actions, lockfiles, pins, runners, release gatesCI/CD: workflows, actions, lockfiles, pins, runners, release gatesmeta:campaignCoordinated multi-repo push with a defined end stateCoordinated multi-repo push with a defined end statepriority:p3Low - nice to haveLow - nice to havescope:estateAffects many or all repos across the estateAffects many or all repos across the estate
Context
membership-integrity(in.github/workflows/anchor-drift.yml) was failing onmainbecause several member submodule gitlinks pointed at commits that no longer exist on the member remotes — orphaned when those member repos' histories were rewritten (squash-merged PRs produce new SHAs). The job aborted with:What was fixed
PR #14 (merged) re-pinned the 6 dangling in-scope members to each member's current
origin/mainHEAD so the recorded pins are reachable again:a2ml-pre-commit,a2ml-deno,a2ml-haskell,a2ml-rs,pandoc-a2ml,vscode-a2mlThe same PR also switched the submodule-resolution step to a full (non-shallow) fetch so non-tip pins resolve.
membership-integrityis now green.Residual risk / why this recurs
Static gitlink pins re-orphan every time a member repo squash-merges or force-pushes, so the gate will silently go red again on the next member history rewrite.
members/tooling/a2mliseris outside the current automation scope (it couldn't be re-pinned in the session that fixed the others) but resolved cleanly in CI.Proposed (ties into #8 — member rollout)
members/*to currentmainand opens a PR, or treatmembership-integrityas the early-warning gate (it now catches this on every PR/push).a2mliserpin under whoever owns that repo.Refs #8.