chore: release gate on merge and marking prerelease tags#598
Conversation
WalkthroughThe release-please workflows are updated to add an explicit ChangesRelease Workflow Triggering and Pre-Release Management
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
Pull request overview
This PR adjusts the repository’s release automation so that release-please only runs on push when the merged commit represents a release, and so that GitHub Releases created/updated by the post-release job are marked as pre-releases when appropriate.
Changes:
- Gate the
release-pleasereusable workflow invocation onpushevents to only run for commits whose message starts withrelease:. - Enhance the post-release job to mark GitHub releases as pre-releases for non-stable versions and to refresh notes on existing releases.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .github/workflows/reusable-release-please.yaml | Adds pre-release flagging logic and refreshes notes for existing GitHub releases during the post-release phase. |
| .github/workflows/release-please.yaml | Adds a job-level condition to run the reusable workflow on push only for release: commits (and always on manual dispatch). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description
This just ensures that release-please runs only on "release: " prefix PR merges or when triggered from workflow dispatch.
Another thing it does is marks beta/alpha/ other experimental tags as prerelease instead of latest like we do in the other SDKs.
What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
main