Repository navigation
Docs: strategy branches, from branch to pull request - #136
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01SACGSprm9GiBueqKdxsy2j
Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01SACGSprm9GiBueqKdxsy2j
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The focused documentation and ignore-rule changes have no identified blocking issues.
Review effort: Balanced
Findings: None
What changed in this PR
Clarifies the strategy contribution workflow and supports shared environments and replay caches in worktrees.
Changes:
- Documents branch naming, CI restrictions, evaluation, and human review and merging.
- Updates ignore rules to cover
.pixiandreplayssymlinks.
| File | Description |
|---|---|
| docs/STRATEGY_DEVELOPMENT.md | Adds the strategy branch-to-pull-request workflow. |
| .gitignore | Ignores .pixi and replays as directories or symlinks. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Match counts no longer assume 22 configs; stage new files before lint; read --reuse's 'to play' line and stop on a stale cache; strategies.md points at round_robin.py --help and says where single-match results go. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01SACGSprm9GiBueqKdxsy2j
| - `pixi run python tools/tournament/round_robin.py [name ...]` — 65s round-robin over every config (or the | ||
| named ones). Flags: `--both-sides`, `--strict`, `--stop-at-first-stall`, | ||
| `--fuzz-restarts SEED`, `--control-scheme {fpp,trajsample,...}`, `--no-save`, `-v`. | ||
| named ones). Common flags: `--reuse`, `--strict`, `--pair A B` (one match); `--help` lists all. |
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
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.
Adds a short "Strategy branches" section to
docs/STRATEGY_DEVELOPMENT.md: branch asstrategy/<idea>, what CI allows it to change (tools/check_strategy_branch.py), what the reviewer checks (no edits to other strategies orpickers.py), and that a person, never an agent, merges. Docs only.🤖 Generated with Claude Code
https://claude.ai/code/session_01SACGSprm9GiBueqKdxsy2j