Re: [edk2-devel] [Patch V2 0/3] Add GitHub IDs to Maintainers.txt

["Michael D Kinney" <michael.d.kinney@intel.com>]

Hi Sean,

Maintainers.txt is a standard file format defined by TianoCore.  Format is in file header.
The intent is for the file format to be consumable by tools and developers.

I reviewed CODEOWERS feature.  It has limitations that do not match the EDK II development process.

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Sean
> Sent: Thursday, July 8, 2021 12:55 PM
> To: devel@edk2.groups.io; Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Leif Lindholm <leif@nuviainc.com>; Feng, Bob C
> <bob.c.feng@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Chen, Christine <yuwei.chen@intel.com>
> Subject: Re: [edk2-devel] [Patch V2 0/3] Add GitHub IDs to Maintainers.txt
> 
> Is Maintainers.txt a standard file used by standard tools or just a
> custom file for the GetMaintainer.py script?
> 
> There is a similar idea used by github (and maybe others) called
> CODEOWNERS.  See
> https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/creating-a-repository-on-github/about-code-
> owners
> 
> This allows github to auto assign reviewers to a PR.  Would it make
> sense to try to align?
> 
> Thanks
> Sean
> 
> 
> 
> On 7/8/2021 12:50 PM, Michael D Kinney wrote:
> > New in V2
> > =========
> > * Maintainers.txt updates
> > * Remove content after email address from standard output
> > * Fix --lookup compatibility with '\' path separators.
> >
> > Update GetMaintainer.py to allow a GitHub ID after the email address
> > for maintainers and reviewers and update Maitainers.txt with GitHub IDs.
> >
> > The GitHub ID will be used to help autotate the PR reviewer assignments.
> >
> > Cc: Andrew Fish <afish@apple.com>
> > Cc: Laszlo Ersek <lersek@redhat.com>
> > Cc: Leif Lindholm <leif@nuviainc.com>
> > Cc: Bob Feng <bob.c.feng@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Yuwei Chen <yuwei.chen@intel.com>
> > Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
> >
> > Michael D Kinney (3):
> >    BaseTools/Scripts: Fix GetMaintainer.py line endings
> >    BaseTools/Scripts: Allow GitHub ID at end Maintainers.txt lines
> >    Maintainers.txt: Add GitHub IDs
> >
> >   BaseTools/Scripts/GetMaintainer.py | 382 +++++++++++++++--------------
> >   Maintainers.txt                    | 282 +++++++++++----------
> >   2 files changed, 331 insertions(+), 333 deletions(-)
> >
> 
> 
> 
>