Thanks Leif, I remove the description from the package name.

Abner

> -----Original Message-----
> From: Leif Lindholm [mailto:[email protected]]
> Sent: Tuesday, August 3, 2021 7:16 PM
> To: [email protected]; Chang, Abner (HPS SW/FW Technologist)
> <[email protected]>
> Cc: Wang, Nickle (HPS SW) <[email protected]>; Liming Gao
> <[email protected]>; Andrew Fish <[email protected]>; Michael D
> Kinney <[email protected]>
> Subject: Re: [edk2-devel] [staging/edk2-redfish-client PATCH v2 2/2] edk2:
> Update Maintainers.txt
> 
> On Tue, Jul 20, 2021 at 11:53:58 +0800, Abner Chang wrote:
> > Add maintainer and reviewer to RedfishClinetPkg.
> >
> > Signed-off-by: Abner Chang <[email protected]>
> > Cc: Nickle Wang <[email protected]>
> > Cc: Liming Gao <[email protected]>
> > Cc: Andrew Fish <[email protected]>
> > Cc: Leif Lindholm <[email protected]>
> > Cc: Michael D Kinney <[email protected]>
> > ---
> >  Maintainers.txt | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/Maintainers.txt b/Maintainers.txt
> > index e9dda5c5ca..55068c9140 100644
> > --- a/Maintainers.txt
> > +++ b/Maintainers.txt
> > @@ -519,6 +519,11 @@ F: RedfishPkg/
> >  M: Abner Chang <[email protected]>
> >  R: Nickle Wang <[email protected]>
> >
> > +RedfishClientPkg: Redfish Client related modules
> 
> My interpretation of the normal use of this field is that we can add a
> comment if the entry applies only to a subset of a package.
> This adds an entry for a whole new package, and as such should only
> state the name of the package.
> 
> With everything from the : until the end of line dropped:
> Reviewed-by: Leif Lindholm <[email protected]>
> 
> /
>     Leif
> 
> > +F: RedfishClientPkg/
> > +M: Abner Chang <[email protected]>
> > +R: Nickle Wang <[email protected]>
> > +
> >  SecurityPkg
> >  F: SecurityPkg/
> >  W: https://github.com/tianocore/tianocore.github.io/wiki/SecurityPkg
> > --
> > 2.17.1
> >
> >
> >
> > 
> >
> >


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#78596): https://edk2.groups.io/g/devel/message/78596
Mute This Topic: https://groups.io/mt/84326489/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to