diff options
| author | Yuriy Andamasov <yuriy@vyos.io> | 2026-05-07 10:32:28 +0300 |
|---|---|---|
| committer | Yuriy Andamasov <yuriy@vyos.io> | 2026-05-07 10:32:28 +0300 |
| commit | a42e5f6828a5b2f640b5e8b64510aa00d0e0e798 (patch) | |
| tree | e53b4510d16b8bd37161fded7554bf6abfae8814 | |
| parent | 09056fc015ebd7e0fc797f29f9510c43591e85da (diff) | |
| download | vyos-documentation-a42e5f6828a5b2f640b5e8b64510aa00d0e0e798.tar.gz vyos-documentation-a42e5f6828a5b2f640b5e8b64510aa00d0e0e798.zip | |
Revert "docs(claude-md): document PR review bot workflow"
The bot review workflow is a cross-repo convention that lives in
the org-level rule file (`vyos-github`-style global instructions),
not in per-repo CLAUDE.md. Documenting it here would duplicate the
canonical source and risk drift if the workflow changes.
This reverts the section added in the previous commit on this branch.
\xf0\x9f\xa4\x96 Generated by [robots](https://vyos.io)
| -rw-r--r-- | CLAUDE.md | 17 |
1 files changed, 0 insertions, 17 deletions
@@ -184,20 +184,3 @@ A PR that both adds and removes TODOs is not contradictory; intent matters. - **Sphinx build** — runs on Read the Docs for every PR; preview URL appears as a check. - **CLA check** — contributors must sign the VyOS CLA before merge. - **Conflict check** — fails the PR if it doesn't merge cleanly into base. - -## PR review workflow - -Two automated reviewers run on PRs in this repo: - -- **Copilot** (`@copilot`) — opt-in, works on draft PRs. Trigger by commenting `@copilot review` on the PR. -- **CodeRabbit** (`@coderabbitai`) — auto-runs when a PR flips to ready-for-review. Does **not** review drafts. Re-trigger with `@coderabbitai review` after each round of fixes. - -Convention: - -1. Open the PR as a **draft** (`gh pr create --draft`). -2. Iterate while draft: comment `@copilot review` per round of commits until Copilot is silent. -3. Flip to ready (`gh pr ready <num>`). CodeRabbit picks it up automatically. -4. Address CodeRabbit threads, push commits, re-comment `@coderabbitai review` if you want another pass. -5. When both bots are silent, the PR is ready for human review. - -Every review thread (Copilot or CodeRabbit) needs an **explicit reply** before resolving — fix in code with the commit SHA, or push back with technical reasoning. Never silently resolve. |
