<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">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>