<div dir="ltr">The +1 means in a general sense "I agree with the fix/feature conveyed at <div>the code and don't see anything that oppose to it being merged". So, I don't </div><div>think that all the times there will be an explanation for the agreement. You</div><div>don't need to rely on +1s, but you can you them as a 'heat' factor that will</div><div>show you how many people care about it and in a way or another have </div><div>reviewed, may be not a thoroughly review, but what they could contribute.<br></div><div><br></div><div>This seems totally harmless and having one more nitpick rule would just</div><div>make new contributor's life harder.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Em ter, 21 de mai de 2019 às 11:31, Marios Andreou <<a href="mailto:marios@redhat.com">marios@redhat.com</a>> escreveu:<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 dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 21, 2019 at 4:34 PM Brian Rosmaita <<a href="mailto:rosmaita.fossdev@gmail.com" target="_blank">rosmaita.fossdev@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">Hello everyone,<br>
<br>
A recent spate of +1 reviews with no comments on patches has put me into<br>
grumpy-old-man mode.<br>
<br>
A +1 with no comments is completely useless (unless you have a review on<br>
a previous patch set with comments that have been addressed by the<br>
author).  I already know you're a smart person (you figured out how to<br>
sign the CLA and navigate gerrit -- lots of people can't or won't do<br>
that), but all your non-comment +1 tells me is that you are in favor of<br>
the patch.  That doesn't give me any information, because I already know<br>
that the author is in favor of the patch, so that makes two of you out<br>
of about 1,168 reviewers.  That's not exactly a groundswell of support.<br>
<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>
<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></blockquote><div><br></div><div>whilst i agree on all you wrote, imo a -1 with no comments is worse than a +1 with no comments. If you dislike my patch enough to -1 it tell me what i need to change in order to fix and get your vote</div><div><br></div><div>thanks, marios</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
That's all I have to say.  I now return to my normal sunny disposition.<br>
<br>
cheers,<br>
brian<br>
<br>
</blockquote></div></div>
</blockquote></div>