<div dir="auto">IMO, if you put +1 to a patch you should put some comments like, verified code as per standards, tested the functionality by applying patch in local environment, works but refactoring is possible.<div dir="auto"><br></div><div dir="auto">The reason behind adding such comments is that it will help to built trust between core and non-core reviewers. This will also help the non-cores to speedup their way to become cores.</div><div dir="auto"><br></div><div dir="auto">Thanks,</div><div dir="auto"><br></div><div dir="auto">Abhishek</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 21 May, 2019, 20:58 Jay Bryant, <<a href="mailto:jungleboyj@gmail.com">jungleboyj@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> When you post your +1, please leave a comment explaining why you<br>
> approve, or at least what in particular you looked at in the patch that<br>
> gave you a favorable impression.  This whole open source community thing<br>
> is a collaborative effort, so please collaborate!  You comment does not<br>
> have to be profound.  Even just saying that you checked that the release<br>
> note or docs on the patch rendered correctly in HTML is very helpful.<br>
I do not leave reviews without some sort of comment.  When I was <br>
mentored into doing reviews the expectation was that you at least leave <br>
some sort of comment with any review.  Also, as Graham noted, especially <br>
for people who are newer to the project this helps give information on <br>
their review.  This is another one of those 'tribal knowledge' items so <br>
I am not going to get too passionate about +1's with or without comments.<br>
> The same thing goes for leaving a -1 on a patch.  Don't just drop a -1<br>
> bomb with no explanation.  The kind of review that will put you on track<br>
> for becoming core in a project is what johnthetubaguy calls a<br>
> "thoughtful -1", that is, a negative review that clearly explains what<br>
> the problem is and points the author in a good direction to fix it.<br>
This obviously is  a requirement and it is just rude to -1 with no <br>
additional direction.<br>
> That's all I have to say.  I now return to my normal sunny disposition.<br>
><br>
> cheers,<br>
> brian<br>
><br>
<br>
</blockquote></div>