Browse files

Clarify light-hearted attitude

  • Loading branch information...
croaky committed Jan 6, 2013
1 parent a70d54a commit 6e44c8c839c66af6645618e2a9fe49dc0876f649
Showing with 3 additions and 3 deletions.
  1. +3 −3 code-review/
@@ -1,7 +1,7 @@
Code Review
-A guide for reviewing code respectfully.
+A guide for reviewing code with a light-hearted and respect attitude.
Read more about the process of reviewing code, in context of writing a feature
or fixing a bug, in the [Protocol](/thoughtbot/guides/blob/master/protocol)
@@ -20,11 +20,11 @@ Everyone
* Be humble. ("I'm not sure - let's look it up.")
* Don't use hyperbole. ("always", "never", "endlessly", "nothing")
* Don't use sarcasm.
+* Keep it real. If emoji, animated gifs, or humor aren't you, don't force them.
+ If they are, use them with aplomb.
* Reach resolutions quickly.
* Talk in person if there are too many "I didn't understand" or "Alternative
solution:" comments. Post a follow-up comment summarizing offline discussion.
-* Use emoji.
-* Use humor.
Author of code for review

0 comments on commit 6e44c8c

Please sign in to comment.