chore(ci): add pre-release lane on develop branch#440
Conversation
WalkthroughA new GitHub Actions workflow ( Changesdevelop Branch Pre-release Workflow and Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #440 +/- ##
=======================================
Coverage 85.74% 85.74%
=======================================
Files 25 25
Lines 1305 1305
Branches 241 265 +24
=======================================
Hits 1119 1119
Misses 108 108
Partials 78 78 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds an opt-in, self-contained pre-release lane for the develop branch using a new GitHub Actions workflow, and documents the intended develop → main promotion model so pre-release release-please commits don’t disrupt stable releases on main.
Changes:
- Added
.github/workflows/release-please-develop.ymlto cut pre-release-only versions fromdevelopvia a shared reusable workflow. - Documented the experimental
developrelease process and the “promote code, not release commits” guidance inRELEASE.md.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| RELEASE.md | Documents the new develop pre-release workflow and branch promotion guidance. |
| .github/workflows/release-please-develop.yml | Introduces a dedicated workflow to create prerelease PRs/tags from develop without altering the stable main release-please lane. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
cbba995 to
0af3af3
Compare
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/release-please-develop.yml:
- Line 41: The reusable workflow reference in the release-please-develop.yml
file uses `@main` which is mutable and creates a supply-chain security risk given
the sensitive permissions and secrets involved. Replace the `@main` reference in
the uses statement for the
openfga/.github/.github/workflows/release-please-prerelease.yml workflow with a
specific immutable commit SHA to ensure the workflow version cannot change
unexpectedly between runs.
In `@RELEASE.md`:
- Around line 96-100: The fenced code block containing the git branch diagram is
missing a language identifier, which triggers the MD040 markdown lint rule. Add
the language identifier `text` to the opening fence of the diagram block by
changing the opening triple backticks to include the language specifier, keeping
the rest of the diagram content unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 60b1f7e0-d22e-4541-85ff-d4f99366fdee
📒 Files selected for processing (2)
.github/workflows/release-please-develop.ymlRELEASE.md
0af3af3 to
1f76072
Compare
Adds .github/workflows/release-please-develop.yml: a standalone workflow_dispatch + push(develop) flow that cuts pre-release-only versions (alpha/beta/rc) via release-please, GPG-signed tags and draft pre-releases. It does NOT modify or call the shared openfga/.github release-please workflow, so the stable main flow is untouched. Publishing is handled by the existing main.yaml tag trigger. Documents the develop-upstream branch model in RELEASE.md.
1f76072 to
6297a01
Compare
|
Making it draft right now |
Adds .github/workflows/release-please-develop.yml: a standalone workflow_dispatch + push(develop) flow that cuts pre-release-only versions (alpha/beta/rc) via release-please, GPG-signed tags and draft pre-releases. It does NOT modify or call the shared openfga/.github release-please workflow, so the stable main flow is untouched. Publishing is handled by the existing main.yaml tag trigger. Documents the develop-upstream branch model in RELEASE.md.
Description
What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit
Documentation
Chores