merge request 187 on GitLab
Rowland Penny
rpenny at samba.org
Thu Jan 17 18:02:11 UTC 2019
On Thu, 17 Jan 2019 09:54:46 -0800
Jeremy Allison via samba-technical <samba-technical at lists.samba.org>
wrote:
> On Thu, Jan 17, 2019 at 03:57:26PM +0100, Martin Krämer via
> samba-technical wrote:
> > Hi all,
> >
> > some days ago I have created a merge request for a samba-tool patch
> > on GitLab.
> > I created this based on instructions from:
> > https://wiki.samba.org/index.php/Samba_CI_on_gitlab
> > and
> > https://wiki.samba.org/index.php/Using_Git_for_Samba_Development
> >
> > Never the less this is my first contribution to samba and my first
> > work with GitLab - I am not sure if I have done everything correct
> > or if there are further actions required by me to be merged.
> >
> > The merge request is:
> > https://gitlab.com/samba-team/samba/merge_requests/187
> >
> > Thanks for instructions
>
> Most of the Samba Team members still look for new patches
> as attachments on the samba-technical mailing list. It's
> always good to send them there as well as on gitlab (which
> we use mostly for CI work, not merging).
>
> Having said that your patch looks good to me !
>
> https://gitlab.com/samba-team/samba/merge_requests/187.patch
>
> Reviewed-by: Jeremy Allison <jra at samba.org>
>
> Can I get a second Team reviewer ?
>
> Thanks a *lot* for helping us with Samba !
>
> Jeremy.
>
+RB
Just don't expect me to push it, we don't want things breaking :)
Rowland
More information about the samba-technical
mailing list