Skip to content

Commit

Permalink
Clarify light-hearted attitude
Browse files Browse the repository at this point in the history
  • Loading branch information
Dan Croak committed Jan 6, 2013
1 parent a70d54a commit 6e44c8c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions code-review/README.md
Original file line number Diff line number Diff line change
@@ -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)
Expand All @@ -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
-------------------------
Expand Down

0 comments on commit 6e44c8c

Please sign in to comment.