I couldn't agree more that code review is a good practice.<div><br></div><div>(I hope it was absolutely clear that I didn't expect you to deny merge requests, it was a theoretical example)<br><br>On Sunday, 29 December 2013, Luiz Irber  wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello, sorry for causing all this problem =]<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 24, 2013 at 9:35 PM, Daniel Narvaez <span dir="ltr"><<a href="javascript:_e({}, 'cvml', 'dwnarvaez@gmail.com');" target="_blank">dwnarvaez@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>On 24 December 2013 23:36, Alan Jhonn Aguiar Schwyn <span dir="ltr"><<a href="javascript:_e({}, 'cvml', 'alanjas@hotmail.com');" target="_blank">alanjas@hotmail.com</a>></span> wrote:<br>

</div><div class="gmail_extra"><div class="gmail_quote"><div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">


<div><div dir="ltr"><div>No. I only aswer this email (see below). </div><div>The mantainer is not "Unresponsive", only takes more of 1 month to asnwer, that what is?</div><div>"slow modem connection user"? :-)</div>


</div></div></blockquote><div><br></div></div><div>Well, to start with give him more of a chance :) He didn't answer to one email, he might have missed it, forgot about it or even thought you didn't need an answer to go ahead and post the merge request.<br>

</div></div></div></div></blockquote><div><br></div><div>About that, maybe I wasn't clear enough in my previous answer, but what I wanted to say was exactly what Daniel said: just go ahead and make a merge request. And yes, I missed this email thread, only today a friend pointed that something was happening.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">

<div>If a maintainer is very slow but it's clear he wants to review code before it's pushed, I don't think we should do so without his permission. It's his project, he is even free to reject perfectly good code without looking at it, if he wants.<br>

</div></div></div></div></blockquote><div><br></div><div>I'm not planning to deny merge requests, but I think code review is a good practice. OK, four years ago we didnt't do it, but nowadays it is important =]</div>

<div><br></div><div>Luiz</div></div></div></div>
</blockquote></div><br><br>-- <br>Daniel Narvaez<br><br>