docs(agents-docs): record the To Be Merged label etiquette (#27089)
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
parent
1883e5c5e0
commit
6501ebe734
1 changed files with 2 additions and 0 deletions
|
|
@ -6,6 +6,8 @@ line, not as background reading.
|
||||||
|
|
||||||
- A posted review carries findings only: no summary, no praise. Nothing to report means one line
|
- A posted review carries findings only: no summary, no praise. Nothing to report means one line
|
||||||
saying so.
|
saying so.
|
||||||
|
- Label `To Be Merged` only on a merge-ready PR, approved or not; remove `Ready to be Reviewed` in
|
||||||
|
the same step.
|
||||||
- A finding names what is wrong, why it matters and the fix, on the offending line. Findings are
|
- A finding names what is wrong, why it matters and the fix, on the offending line. Findings are
|
||||||
bugs, crashes, lifetime and memory errors, data-integrity problems, injection, and violations of
|
bugs, crashes, lifetime and memory errors, data-integrity problems, injection, and violations of
|
||||||
a rule written down in AGENTS.md or `.agents/docs/`; taste no rule covers is not a finding.
|
a rule written down in AGENTS.md or `.agents/docs/`; taste no rule covers is not a finding.
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue