Mesa (master): docs: Add an encouraging note about providing reviews and acks.

[GitLab Mirror]

Module: Mesa
Branch: master
Commit: 90818558f0e8e50cd20f637775f072b5511edfd0
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=90818558f0e8e50cd20f637775f072b5511edfd0

Author: Eric Anholt <eric at anholt.net>
Date:   Wed Dec 12 11:11:07 2018 -0800

docs: Add an encouraging note about providing reviews and acks.

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 at pnwbakers.com>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

---

 docs/submittingpatches.html | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/docs/submittingpatches.html b/docs/submittingpatches.html
index d0b61a3409..61c6d1f880 100644
--- a/docs/submittingpatches.html
+++ b/docs/submittingpatches.html
@@ -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>