From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id DBCAB2095DE5A for ; Thu, 10 Aug 2017 15:09:30 -0700 (PDT) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga105.jf.intel.com with ESMTP; 10 Aug 2017 15:11:50 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.41,355,1498546800"; d="scan'208";a="122257928" Received: from dtmedina-mobl.amr.corp.intel.com (HELO jljusten-skl.amr.corp.intel.com) ([10.254.186.35]) by orsmga002.jf.intel.com with ESMTP; 10 Aug 2017 15:11:50 -0700 From: Jordan Justen To: edk2-devel@lists.01.org Cc: Jordan Justen , Andrew Fish , Leif Lindholm , Michael D Kinney Date: Thu, 10 Aug 2017 15:11:29 -0700 Message-Id: <20170810221129.7383-3-jordan.l.justen@intel.com> X-Mailer: git-send-email 2.14.0 In-Reply-To: <20170810221129.7383-1-jordan.l.justen@intel.com> References: <20170810221129.7383-1-jordan.l.justen@intel.com> Subject: [PATCH 3/3] Maintainers.txt: Add Package Reviewer role X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 10 Aug 2017 22:09:31 -0000 Although everyone is encouraged to review patches and add their Reviewed-by reply for a patch, with the Package Reviewer role we identify additional community members that will be Cc'd for patches made to a package. A distinction between a Package Maintainer and Reviewer is that Maintainers will always have source control push access to the package whereas Reviewers will not. (The Reviewer may have push access if they are also Package Maintainer for another package.) Currently we have an limit of 2 Package Maintainers per package, but the Package Maintainers for each package decide how to manage the Package Reviewer list. Cc: Andrew Fish Cc: Leif Lindholm Cc: Michael D Kinney Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Jordan Justen --- Maintainers.txt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/Maintainers.txt b/Maintainers.txt index 92a1fd974f..6317a392e7 100644 --- a/Maintainers.txt +++ b/Maintainers.txt @@ -5,7 +5,8 @@ This file provides information about the primary maintainers for EDK II. In general, you should not privately email the maintainer. You should -email the edk2-devel list, but you can also Cc the maintainer. +email the edk2-devel list, and Cc the package maintainers and +reviewers. Descriptions of section entries: @@ -13,6 +14,8 @@ Descriptions of section entries: Patches and questions should be sent to the email list. M: Package Maintainer: Cc address for patches and questions. Responsible for reviewing and pushing package changes to source control. + R: Package Reviewer: Cc address for patches and questions. Reviewers help + maintainers review code, but don't have push access. W: Web-page with status/info T: SCM tree type and location. Type is one of: git, svn. S: Status, one of the following: -- 2.14.0