Thanks Leif, I remove the description from the package name. Abner
> -----Original Message----- > From: Leif Lindholm [mailto:l...@nuviainc.com] > Sent: Tuesday, August 3, 2021 7:16 PM > To: devel@edk2.groups.io; Chang, Abner (HPS SW/FW Technologist) > <abner.ch...@hpe.com> > Cc: Wang, Nickle (HPS SW) <nickle.w...@hpe.com>; Liming Gao > <gaolim...@byosoft.com.cn>; Andrew Fish <af...@apple.com>; Michael D > Kinney <michael.d.kin...@intel.com> > 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 <abner.ch...@hpe.com> > > Cc: Nickle Wang <nickle.w...@hpe.com> > > Cc: Liming Gao <gaolim...@byosoft.com.cn> > > Cc: Andrew Fish <af...@apple.com> > > Cc: Leif Lindholm <l...@nuviainc.com> > > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > --- > > 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 <abner.ch...@hpe.com> > > R: Nickle Wang <nickle.w...@hpe.com> > > > > +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 <l...@nuviainc.com> > > / > Leif > > > +F: RedfishClientPkg/ > > +M: Abner Chang <abner.ch...@hpe.com> > > +R: Nickle Wang <nickle.w...@hpe.com> > > + > > 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: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
