Strip release-process narration from per-repo docs - #144
Conversation
The release process is owned by rainix's rainix-tag-release reusable and restating it per-repo drifts: rainlanguage/rainix#338 made tag-release push-free, so the 'commits the snapshot back to main' narration here was already stale. Docs-only; repo-local invariants kept, mechanics point to rainix. Ruling: rainlanguage/claude-audit-skills#91 Philosophy: rainlanguage/rainix#298 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe release workflow comment and README now provide concise documentation for manual ChangesRelease documentation
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This docs-only change removes duplicated release-process details without changing repository behavior; no actionable merge-blocking risk remains after normal checks and review. Possibly related issues
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Docs-only. Strip release-process narration that this per-repo doc restates but does not own.
Human ruling (2026-08-20): "you really shouldn't put release processes in the per repo docs because it's super fragile." The release process is owned by rainix's
rainix-tag-release.yamlreusable; per-repo restatements drift the moment the reusable changes — which just happened: rainlanguage/rainix#338 / rainlanguage/rainix#339 made tag-release push-free, so this repo's "commits the frozen snapshot back to main" was already factually wrong.Removed
.github/workflows/package-release.yaml— collapsed the ~23-line header comment (rainix-tag-release runs cutRelease / freezes + regenerates the released-suites lib / verifies chains / publishes to Soldeer / commits both back to main, plus the why-not-autopublish lifecycle rationale) to one line of deploy-repo identity.README.md"Deploying, and then releasing" step 3 — removed therainix-tag-releasemechanics (regenerate snapshot / verify chains / publish to Soldeer / commit frozen snapshot back to main — now stale).Kept (repo-local facts / product docs)
The three-step deploy→verify→tag ordering and why it holds (deploy is idempotent, key-custody, workflow_dispatch-only; verify is scoped to released suites; a tag verifies and publishes but never broadcasts, which is why the deploy cannot fold into it); the lifecycle paragraph (
[external.package].versionis the last publish, historical versions unaffected).CLAUDE.mdis unchanged — its release bullet already carries only repo-local invariants, no shared-workflow mechanics.cutRelease()docs elsewhere are left intact: it is THIS repo's own product code, not shared-workflow narration.QA
Summary by CodeRabbit
sol-v*tags.