Skip to content

Commit 0478de3

Browse files
committed
Disclaim if you haven't fully tested a suggestion
Clarify if you haven't tested a code suggestion or got an information from an unofficial source like an LLM or blog post.
1 parent 4dac788 commit 0478de3

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

code-review/README.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,11 @@ Watch a presentation that covers this material from [Derek Prior at RailsConf 20
3636
- **Be humble**
3737
- "I'm not sure - let's look it up."
3838

39+
- **Disclaim if you haven't fully tested a suggestion**
40+
- "I think we can rewrite this as \<suggestion\>, but I haven't tested it."
41+
- This applies to code and information in general.
42+
- Be cautious with information you didn't get from official sources.
43+
3944
- **Don't use hyperbole**
4045
- "Always", "never", "endlessly", "nothing"
4146

0 commit comments

Comments
 (0)