docs: Add an encouraging note about providing reviews and acks.
authorEric Anholt <eric@anholt.net>
Wed, 12 Dec 2018 19:11:07 +0000 (11:11 -0800)
committerEric Anholt <eric@anholt.net>
Wed, 19 Dec 2018 20:49:17 +0000 (12:49 -0800)
Across several projects I've seen new contributors say "I wasn't sure if I
should provide a review tag since I'm not really an expert in this area."
Everyone I know already applies some implicit weighting to reviews from
different people, so encourage participation.

Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
docs/submittingpatches.html

index d0b61a340992b64617f9352d0b567d735125741f..61c6d1f88060c5d01d42a40a1b8eaf069f1fb593 100644 (file)
@@ -302,6 +302,14 @@ These Reviewed-by, Acked-by, and Tested-by tags should also be amended
 into commits in a MR before it is merged.
 </p>
 
+<p>
+Review by non-experts is encouraged.  Understanding how someone else
+goes about solving a problem is a great way to learn your way around
+the project.  The submitter is expected to evaluate whether they have
+an appropriate amount of review feedback from people who also
+understand the code before merging their patches.
+</p>
+
 <h2 id="nominations">Nominating a commit for a stable branch</h2>
 
 <p>