Repository navigation
fix(release): configure Git identity for beta finalize - #246
Conversation
|
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. 📝 WalkthroughWalkthroughThe beta release workflow now configures Git identity during FINALIZE as well as PREPARE when projects are selected. Release-policy contract tests validate the required modes, project condition, and regression behavior. ChangesBeta release Git identity
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This change enables Git attribution for eligible PREPARE and FINALIZE release runs while preserving existing project, authorization, SHA, and concurrency gates. No actionable merge-blocking risk remains beyond normal checks and review. Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Linked Issues checkExplanation The changes satisfy the coding objectives in [ Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Closes #245
Type
Summary
Configure the GitHub Actions bot Git identity for both project-bearing PREPARE and FINALIZE modes. FINALIZE creates annotated tags, so it must have a tagger identity before
git tag -aruns.Root cause
Authorized FINALIZE run 33205123705 reached the exact merged beta path but failed with exit code 128 before creating any public artifact. Git identity setup was gated to PREPARE only.
Changes
prepareorfinalize.Test plan
git diff --checkpass.Safety
masterSHA.Summary by CodeRabbit
Release Improvements
Bug Fixes