<div dir="ltr">Hello Patrick,<div><br></div><div>First a big thank you ! It is always great to have external contribution and I agree to say that our process shoudl be explained more clearly.<br><div><br></div><div>If you think your Merge Request is ready to be tested and merged, please tag a VTK developpers on it so we can review it.</div><div>Please read the VTK coding standard to be sure that your code is following it.</div><div><a href="https://docs.google.com/document/d/1nzinw-dR5JQRNi_gb8qwLL5PnkGMK2FETlQGLr10tZw/edit">https://docs.google.com/document/d/1nzinw-dR5JQRNi_gb8qwLL5PnkGMK2FETlQGLr10tZw/edit</a><br></div><div><br></div><div>The easiest to know how to tag could be by using git blame on the file you are modifying.</div><div>In any case you can always tag : @ken\-martin @berkgeveci @mwestphal @jpouderoux @allisonvacanti @demarle @dgobbi @cory.quammen @acbauer </div><div>Just to name a few.</div><div>You can also send an email on this mailing list asking for reviews and tests, while describing a little bit so people quickly reading their mail can know if they are interested or not.<br></div><div><br></div><div><div style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">Once you have a total of "+2" on your MR, and tests passes, you can ask for a merge, and we will take care of it.<br></div><br></div><div><br></div><div>Best regards,</div><div><br></div><div>For those how want to take a look, here are Patrick's contributions</div><div><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/4451">https://gitlab.kitware.com/vtk/vtk/merge_requests/4451</a><br></div><div><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/4452">https://gitlab.kitware.com/vtk/vtk/merge_requests/4452</a><br></div><div><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/4461">https://gitlab.kitware.com/vtk/vtk/merge_requests/4461</a><br></div><div><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/4462">https://gitlab.kitware.com/vtk/vtk/merge_requests/4462</a><br></div><div><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/4463">https://gitlab.kitware.com/vtk/vtk/merge_requests/4463</a><br></div><div><br></div><div><br></div><div><br></div><div><br></div></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Mathieu Westphal<br></div></div></div>
<br><div class="gmail_quote">On Thu, Jul 5, 2018 at 2:15 PM, Patrick Bergeron <span dir="ltr"><<a href="mailto:pbergeron@spiria.com" target="_blank">pbergeron@spiria.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I am wondering how was he merge request process works. There are about 140 mrs today. <br>
<br>
How do they get approved or declined? Do they stay in the queue forever? When do submitters get feedback about the mrs?<br>
<br>
Thanks<br>
Patrick Bergeron<br>
<br>
Sent from my iPhone<br>
______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="https://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">https://public.kitware.com/<wbr>mailman/listinfo/vtk-<wbr>developers</a><br>
<br>
</blockquote></div><br></div>