docs: route breaking changes and major features to a dev branch#1522
Conversation
Add a Branch Targeting section explaining when to base a PR on `main` vs `dev`, and note that CI already runs on PRs against either branch (no workflow changes needed since `pull_request:` triggers have no `branches:` filter). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Actionable comments posted: 0 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThis PR updates 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches✨ Simplify code
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 |
Summary
Branch Targetingsection toCLAUDE.mddescribing when contributors should base PRs onmain(stable: bug fixes, small additive changes, docs, behavior-preserving refactors) vs the newdevbranch (wire-protocol changes, breaking API changes in FFI / libmoq / moq-net / hang / language wrappers, catalog/container format changes, major features that need to settle).Workflowchecklist so contributors pick a base branch before they start.origin/devhas been (re-)created fromorigin/mainto host this staging branch.Why
We want a place to land disruptive work in batches without holding up small PRs against
main, and without each disruptive PR sitting open onmainfor a long review cycle.CI
No workflow changes are needed. Every
pull_request:trigger in.github/workflows/(check.yml,moq-relay.yml,moq-cli.yml,moq-token-cli.yml,libmoq.yml,check-swift.yml) is configured without abranches:filter, so PRs targetingdevalready run the same CI as PRs targetingmain. The only branch-scoped triggers arepush: branches: [main]onrelease-rs.ymlandrelease-js.yml(release-plz), which intentionally stay scoped tomain.Test plan
devand confirmcheck, path-filtered Rust workflows, andcheck-swiftall fire as expected.release-rs.yml/release-js.ymldo not fire on pushes todev.(Written by Claude)