Maintainers.txt: clarify Reviewer requirements and responsibilities

The current language for "Package Reviewer" only vaguely hints that
Package Reviewers should be able to provide guidance and directions.
Make this more obvious.

Cc: Andrew Fish <afish@apple.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Philippe Mathieu-Daude <philmd@redhat.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Reviewed-by: Andrew Fish <afish@apple.com>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>

diff --git a/Maintainers.txt b/Maintainers.txt
index 61c57587a66b8721057f78112d10fa4c4f9573b3..be77898ee2f100f2ebbe61ac97533948fecea0ba 100644 (file)
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -20,7 +20,10 @@ Descriptions of section entries:
   M: Package Maintainer: Cc address for patches and questions. Responsible\r
      for reviewing and pushing package changes to source control.\r
   R: Package Reviewer: Cc address for patches and questions. Reviewers help\r
-     maintainers review code, but don't have push access.\r
+     maintainers review code, but don't have push access. A designated Package\r
+     Reviewer is reasonably familiar with the Package (or some modules\r
+     thereof), and/or provides testing or regression testing for the Package\r
+     (or some modules thereof), in certain platforms and environments.\r
   W: Web-page with status/info\r
   T: SCM tree type and location.  Type is one of: git, svn.\r
   S: Status, one of the following:\r