Hi Dear VPP Would you please take a look at https://gerrit.fd.io/r/#/c/20325/ and merge this commit. previously Damjan told me to amend my commit and add nat in MAINTAINERS file but I did it in another commit independently.
Thanks Chore On Tue, Jun 25, 2019 at 5:01 PM Damjan Marion <[email protected]> wrote: > Dear Emma, > > This is exact reason why we introduced commit checks. It will help us to > keep MAINTAINERS file up to date, without this check nobody will notice > that nat is missing there. > > Ole is our nat guy, please amend your patch with MAINTAINERS update and > add Ole as maintainer... > > — > Damjan > > > On Jun 25, 2019, at 2:20 PM, emma sdi <[email protected]> wrote: > > > > Hi Dear VPP > > > > Why NAT dose not exist in MAINTAINERS file?! as you know, during build > phase, commit message subject is inspected inside MAINTAINERS file. so how > could be possible a succeeded commit in NAT plug-in? > > > > Thanks > > Chore > > -=-=-=-=-=-=-=-=-=-=-=- > > Links: You receive all messages sent to this group. > > > > View/Reply Online (#13362): https://lists.fd.io/g/vpp-dev/message/13362 > > Mute This Topic: https://lists.fd.io/mt/32202722/675642 > > Group Owner: [email protected] > > Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [[email protected]] > > -=-=-=-=-=-=-=-=-=-=-=- > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#13375): https://lists.fd.io/g/vpp-dev/message/13375 Mute This Topic: https://lists.fd.io/mt/32202722/21656 Group Owner: [email protected] Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
