On Mon, Jul 8, 2024 at 5:05 PM Dumitru Ceara <[email protected]> wrote:
>
> On 7/8/24 22:38, [email protected] wrote:
> > From: Numan Siddique <[email protected]>
> >
> > Also added my ovn.org email id to the list.
> >
> > Signed-off-by: Numan Siddique <[email protected]>
> > ---
>
> Hi Numan,
>
> >  MAINTAINERS.rst | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/MAINTAINERS.rst b/MAINTAINERS.rst
> > index f72c8b5eda..0d70048078 100644
> > --- a/MAINTAINERS.rst
> > +++ b/MAINTAINERS.rst
> > @@ -54,7 +54,8 @@ This is the current list of active OVN committers:
> >     * - Mark Michelson
> >       - [email protected]
> >     * - Numan Siddique
> > -     - [email protected]
> > +     - [email protected]
> > +     - [email protected]
>
> Unfortunately, this breaks the format of the maintainers file because
> the table only has two columns and this adds 3 values for this row:
>
> https://github.com/ovsrobot/ovn/blob/series_414329/MAINTAINERS.rst
>
> Maybe you can change it to list both emails, separated by comma?  I

Thanks for pointing out this error.
Separated by comma seems fine to me. Let me test it out before submitting v2.

Thanks
Numan

> don't really know much .rst so maybe there are other options too.
>
> Regards,
> Dumitru
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to