Merge pull request #864 from public/code-review-docs
Make code review docs friendlier
diff --git a/docs/development/reviewing-patches.rst b/docs/development/reviewing-patches.rst
index 9147a73..bd3ee96 100644
--- a/docs/development/reviewing-patches.rst
+++ b/docs/development/reviewing-patches.rst
@@ -1,8 +1,11 @@
-Reviewing/merging patches
-=========================
+Reviewing and merging patches
+=============================
-Everyone is encouraged to review open pull requests. When reviewing a patch try
-to keep each of these concepts in mind:
+Everyone is encouraged to review open pull requests. We only ask that you try
+and think carefully, ask questions and are `excellent to one another`_. Code
+review is our opportunity to share knowledge, design ideas and make friends.
+
+When reviewing a patch try to keep each of these concepts in mind:
Architecture
------------
@@ -24,8 +27,8 @@
* Has it been documented?
* Will this change introduce new bugs?
-Grammar/Style
--------------
+Grammar and style
+-----------------
These are small things that are not caught by the automated style checkers.
@@ -54,3 +57,4 @@
The purpose of these policies is to minimize the chances we merge a change
that jeopardizes our users' security.
+.. _`excellent to one another`: https://speakerdeck.com/ohrite/better-code-review