public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Michael D Kinney" <michael.d.kinney@intel.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	"rfc@edk2.groups.io" <rfc@edk2.groups.io>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Subject: GitHub PR code review, new EDK II Reviewers team, and the edk2-codereview repository
Date: Fri, 9 Sep 2022 02:06:22 +0000	[thread overview]
Message-ID: <CO1PR11MB4929FA7FB052DCB52FCAA68BD2439@CO1PR11MB4929.namprd11.prod.outlook.com> (raw)

Hello,

I am exploring a proposal to use GitHub PR for code review.  Specifically
the last option listed in the following discussion:

    https://github.com/tianocore/edk2/discussions/3017

I am using the edk2-codereview repository to try different ideas out
and to get feedback from the community on the tools and review process.
It has all the standard CI checks enabled along with extra GitHub 
Actions to assign reviewers and to verify the consistency of the 
Maintainers.txt, CODEOWNERS, and REVIEWERS files.

    https://github.com/tianocore/edk2-codereview

Part of that proposal is assignment of maintainers and reviewers to a PR
and use of the CODEOWNERS feature of GitHub for maintainers and adding a
second REVIEWERS file that uses the same CODEOWNERS syntax for reviewers.

In order to make a review request all maintainers and reviewers
must be members of the TianoCore organization.  We already have a
team called "EDK II Maintainers" for maintainers.  I have added a
new team called "EDK II Reviewers" for the set of reviewers that 
are not maintainers of any other edk2 packages/directory.

If you received an invite to join the "EDK II Reviewers" team,
then please accept that invitation.

The GitHub Action CI checks I am enabling as part of this proposal
are getting errors because some reviewers are not part of the 
TianoCore organization yet.

Example GitHub Action CI failure in edk2-codereview repository:

https://github.com/tianocore/edk2-codereview/runs/8259406920?check_suite_focus=true

As sample PRs are opened in edk2-codereview, you may receive email
notices.  Please explore the PRs and provide feedback on the tools and
the process.

You are also welcome to take any of the active code reviews on the 
mailing list and submit them to the edk2-codereview repository to
try out the GitHub PR code review experience with real examples.

Please hold off on merging PRs into edk2-codereview repo for now.

There are no changes to the dev process for the edk2 repo at this time.

Thanks,

Mike


                 reply	other threads:[~2022-09-09  2:06 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CO1PR11MB4929FA7FB052DCB52FCAA68BD2439@CO1PR11MB4929.namprd11.prod.outlook.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox