<div dir="ltr"><div class="gmail_default" style="font-size:small">I meant I totally agree with what you're saying. If a core has already put a +2 on a patch, the next core can approve.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Apr 11, 2019 at 4:51 PM Trinh Nguyen <<a href="mailto:dangtrinhnt@gmail.com">dangtrinhnt@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small">Ok, that makes sense to me.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Apr 11, 2019 at 4:50 PM Rong Zhu <<a href="mailto:aaronzhu1121@gmail.com" target="_blank">aaronzhu1121@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div dir="auto">Hi,</div></div><div dir="auto"><br></div><div dir="auto">As usual, There are have +2s, then the next core can +2/A, why we need PTL final +A?</div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto">Rong Zhu</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Trinh Nguyen <<a href="mailto:dangtrinhnt@gmail.com" target="_blank">dangtrinhnt@gmail.com</a>>于2019年4月11日 周四15:39写道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi team,</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">One of the thing that has been brought up in the last meeting is what is the process of merging a patch. So, let's try this first:</div><div class="gmail_default" style="font-size:small"><ul><li>For small patches (e.g., change strings, fix typos, update docs): any core can approve for merging.</li><li>For a patch that implements a new feature, removes a feature, patch that has huge changes (in the number of changed files and loc, etc.) or any patch that is in doubt: we need at least 2 cores to give a +2 and then the PTL will merge it.</li></ul><div>Of course, we can change the procedure when we see issues. For now, this should be a guideline for the team.</div><div><br></div><div>Please feel free to comment.</div><div><br></div><div>Yours,</div></div></div><div dir="ltr"><div><br></div>-- <br><div dir="ltr" class="gmail-m_-8303496425443939686gmail-m_-1606202128722695594m_429042436412510714gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><b style="font-size:small;color:rgb(51,51,51)">Trinh Nguyen</b><br></div><div><u style="font-size:12.8px;color:rgb(0,0,0)"><a href="https://www.edlab.xyz" target="_blank">www.edlab.xyz</a></u><br></div><div><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>
</blockquote></div></div>-- <br><div dir="ltr" class="gmail-m_-8303496425443939686gmail-m_-1606202128722695594gmail_signature">Thanks,<br>Rong Zhu</div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail-m_-8303496425443939686gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><b style="font-size:small;color:rgb(51,51,51)">Trinh Nguyen</b><br></div><div><u style="font-size:12.8px;color:rgb(0,0,0)"><a href="https://www.edlab.xyz" target="_blank">www.edlab.xyz</a></u><br></div><div><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><b style="font-size:small;color:rgb(51,51,51)">Trinh Nguyen</b><br></div><div><u style="font-size:12.8px;color:rgb(0,0,0)"><a href="https://www.edlab.xyz" target="_blank">www.edlab.xyz</a></u><br></div><div><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>