diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-11 01:56:07 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-11 01:56:07 -0500 |
| commit | 99bd213002f2bd24192d15ff9ae69a0379b2f5be (patch) | |
| tree | 27ae8c0fa517ae65b8e68a006cb43c862bad55db /add-tests | |
| parent | 81ca16f9b516112ae17727a09b8b36538f9db21d (diff) | |
| download | rulesets-99bd213002f2bd24192d15ff9ae69a0379b2f5be.tar.gz rulesets-99bd213002f2bd24192d15ff9ae69a0379b2f5be.zip | |
docs(review): drop praise from approve summaries, print inline text at the gate
An approve summary now carries no praise, not even a bare positive like "Clean." or "Solid fix." Lead with the substantive pointer (the design note pinned inline) and close with the verdict. This removes the bare-positive carve-out from review-code's Posted Summary Voice and from voice pattern #40, and updates the worked examples.
The review gate must also print the full prose of every inline comment, never a description of it. The exact words post under my name, so I have to see them before I approve. Made explicit in review-code Phase 5 and commits.md Shape 1.
From a DeepSat review session where I approved "One design note inline, not a blocker. Approving."
Diffstat (limited to 'add-tests')
0 files changed, 0 insertions, 0 deletions