docs: add explicit no-PR rule for maintainer repos
Polecats were creating GitHub PRs instead of using gt done to submit to the Refinery. Added clear conditional language: - If repo is steveyegge/beads or steveyegge/gastown: NEVER create PRs - Polecats use gt done → Refinery merges - Crew workers push directly to main - PRs are for external contributors only This fixes a prompting gap that led to PR #292 being created incorrectly. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -222,10 +222,17 @@ instructions immediately. Useful for one-off tasks that don't warrant a full bea
|
|||||||
|
|
||||||
## Git Workflow: Work Off Main
|
## Git Workflow: Work Off Main
|
||||||
|
|
||||||
**Crew workers push directly to main. No feature branches. NEVER create PRs.**
|
**Crew workers push directly to main. No feature branches.**
|
||||||
|
|
||||||
PRs are for external contributors submitting changes for review. As crew, you have
|
### No PRs in Maintainer Repos
|
||||||
direct commit access - use it. If you create a PR, you're adding unnecessary overhead.
|
|
||||||
|
If the remote origin is `steveyegge/beads` or `steveyegge/gastown`:
|
||||||
|
- **NEVER create GitHub PRs** - you have direct push access
|
||||||
|
- Crew workers: push directly to main
|
||||||
|
- Polecats: use `gt done` → Refinery merges to main
|
||||||
|
|
||||||
|
PRs are for external contributors submitting to repos they don't own.
|
||||||
|
Check `git remote -v` if unsure about repo ownership.
|
||||||
|
|
||||||
### The Landing Rule
|
### The Landing Rule
|
||||||
|
|
||||||
|
|||||||
@@ -236,6 +236,16 @@ to the merge queue. The Witness handles the rest.
|
|||||||
**Note:** Do NOT manually close the root issue with `bd close`. The Refinery
|
**Note:** Do NOT manually close the root issue with `bd close`. The Refinery
|
||||||
closes it after successful merge. This enables conflict-resolution retries.
|
closes it after successful merge. This enables conflict-resolution retries.
|
||||||
|
|
||||||
|
### No PRs in Maintainer Repos
|
||||||
|
|
||||||
|
If the remote origin is `steveyegge/beads` or `steveyegge/gastown`:
|
||||||
|
- **NEVER create GitHub PRs** - you have direct push access
|
||||||
|
- Polecats: use `gt done` → Refinery merges to main
|
||||||
|
- Crew workers: push directly to main
|
||||||
|
|
||||||
|
PRs are for external contributors submitting to repos they don't own.
|
||||||
|
Check `git remote -v` if unsure about repo ownership.
|
||||||
|
|
||||||
### The Landing Rule
|
### The Landing Rule
|
||||||
|
|
||||||
> **Work is NOT landed until it's on `main` OR in the Refinery MQ.**
|
> **Work is NOT landed until it's on `main` OR in the Refinery MQ.**
|
||||||
|
|||||||
@@ -162,6 +162,16 @@ The `gt done --exit` command:
|
|||||||
- Exits your session immediately (no idle waiting)
|
- Exits your session immediately (no idle waiting)
|
||||||
- Witness handles cleanup, Refinery merges your branch
|
- Witness handles cleanup, Refinery merges your branch
|
||||||
|
|
||||||
|
### No PRs in Maintainer Repos
|
||||||
|
|
||||||
|
If the remote origin is `steveyegge/beads` or `steveyegge/gastown`:
|
||||||
|
- **NEVER create GitHub PRs** - you have direct push access
|
||||||
|
- Polecats: use `gt done` → Refinery merges to main
|
||||||
|
- Crew workers: push directly to main
|
||||||
|
|
||||||
|
PRs are for external contributors submitting to repos they don't own.
|
||||||
|
Check `git remote -v` if unsure about repo ownership.
|
||||||
|
|
||||||
### The Landing Rule
|
### The Landing Rule
|
||||||
|
|
||||||
> **Work is NOT landed until it's on `main` OR in the Refinery MQ.**
|
> **Work is NOT landed until it's on `main` OR in the Refinery MQ.**
|
||||||
|
|||||||
Reference in New Issue
Block a user