docs: record that PRs target dev, not main - #571
Merged
Conversation
CLAUDE.md described no branch or PR workflow, and the one place that touched on it told contributors to open a PR off `main`/`dev`. In practice `dev` is the integration branch and `main` trails it by around a hundred commits, so `main` is not a viable base. Getting this wrong is not merely cosmetic: test-helper conventions move quickly on `dev`, so a branch cut from an older base can carry tests that no longer run there at all. Say so, and say to rebase first.
azfoo
approved these changes
Aug 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CLAUDE.md described no branch or PR workflow at all, and the one line that touched
on it — in Commit hygiene — told contributors to open a separate PR "off
main/dev", implyingmainis a valid base.In practice every recent PR (#555–#562, #570) targets
dev, andmaintrails it byaround a hundred commits, so
mainis not a viable base.This is not merely cosmetic. Test-helper conventions move quickly on
dev: a branchcut from
mainor from an older base can carry tests that reference helpers which nolonger exist —
tlui.create_hierarchy(...)has been replaced bycommands.execute("timeline.hierarchy.add", start=..., end=..., level=...)— so agreen local run says nothing about whether the tests even start on
dev.Adds a short Branches and PRs section stating the base branch and the
rebase-first rule, and tightens the Commit hygiene line to just
dev.Docs only; no code or test changes.
🤖 Generated with Claude Code