aboutsummaryrefslogtreecommitdiff
path: root/scripts/sync-language-bundle.sh
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-05-22 13:48:59 -0500
committerCraig Jennings <c@cjennings.net>2026-05-22 13:48:59 -0500
commita4389e8259a68d6e237485f6de4f05c7d1c6cc5d (patch)
tree054b9980829b135c8ca701ab23b712f45f73435e /scripts/sync-language-bundle.sh
parenta785f54eb4f8e45f97be737047eb5a4547a5dd5a (diff)
downloadrulesets-a4389e8259a68d6e237485f6de4f05c7d1c6cc5d.tar.gz
rulesets-a4389e8259a68d6e237485f6de4f05c7d1c6cc5d.zip
docs(skills): keep review-code praise honest and unforced
Two related changes to review-code's strengths guidance. The mandatory "three minimum" could force filler on a tiny diff or padded praise on a weak PR, so I relaxed it to up to three specific strengths, with an honest "nothing notable" allowed when the diff doesn't earn them. I also reframed the old "No Strengths section" anti-pattern as "skipping strengths out of laziness": a substantive diff still demands them, a weak one doesn't. The other change tells reviewers to name the good thing and stop, without explaining why it's good. Explaining praise reads as sycophantic since the author already knows the rationale. Elaboration is for findings, not compliments.
Diffstat (limited to 'scripts/sync-language-bundle.sh')
0 files changed, 0 insertions, 0 deletions