diff --git a/skills/getting-a-code-review/SKILL.md b/skills/getting-a-code-review/SKILL.md index 6ef097d3b28de3fcb3ae8ad2618f0690ade43887..c2fe0f23412c6ebae8b5a274d2961fa8c6b34c19 100644 --- a/skills/getting-a-code-review/SKILL.md +++ b/skills/getting-a-code-review/SKILL.md @@ -38,7 +38,9 @@ amp review -f src/auth.ts amp review "068d2222..1800e595" -i "explicit, detailed instructions to focus on something(s) in particular" -f src/auth.ts ``` -NEVER set a timeout when running these commands. +NEVER set a timeout when running these commands. A reviewer timing out is a disastrous waste of your time and the reviewer's! + +Some of them require running in git repos. Some of them don't understand jujutsu refs and require colocation in a git repo along with "ref..ref" ranges using the commit IDs instead of jujutsu changes. Most, I believe, are able to review unstaged changes in a colocated/git repo. ## Output