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.kin...@intel.com> > Cc: Andrew Fish <af...@apple.com>; Laszlo Ersek <ler...@redhat.com>; Leif > Lindholm <l...@nuviainc.com>; Feng, Bob C > <bob.c.f...@intel.com>; Liming Gao <gaolim...@byosoft.com.cn>; Chen, > Christine <yuwei.c...@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 <af...@apple.com> > > Cc: Laszlo Ersek <ler...@redhat.com> > > Cc: Leif Lindholm <l...@nuviainc.com> > > Cc: Bob Feng <bob.c.f...@intel.com> > > Cc: Liming Gao <gaolim...@byosoft.com.cn> > > Cc: Yuwei Chen <yuwei.c...@intel.com> > > Signed-off-by: Michael D Kinney <michael.d.kin...@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(-) > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#77617): https://edk2.groups.io/g/devel/message/77617 Mute This Topic: https://groups.io/mt/84076273/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
