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 <s3m2e1.6s...@gmail.com> 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: vpp-dev+ow...@lists.fd.io > Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [dmar...@me.com] > -=-=-=-=-=-=-=-=-=-=-=-
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#13363): https://lists.fd.io/g/vpp-dev/message/13363 Mute This Topic: https://lists.fd.io/mt/32202722/21656 Group Owner: vpp-dev+ow...@lists.fd.io Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
