<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<div>Ive seen this a lot too. It is frustrating.<br>
<br>
Kevin <strong>
<div><font face="Tahoma" color="#000000" size="2"> </font></div>
</strong>
<hr tabindex="-1">
<font face="Tahoma" size="2"><b>From:</b> Julien Danjou<br>
<b>Sent:</b> Friday, April 24, 2015 1:14:38 AM<br>
<b>To:</b> openstack-dev@lists.openstack.org<br>
<b>Subject:</b> [openstack-dev] Please stop reviewing code while asking questions<br>
</font><br>
<div></div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Hi there,<br>
<br>
This is now happening weekly to me now, probably because I write too<br>
many patches touching almost all OpenStack projects once a cycle, and<br>
I'm really tired of that behavior, so PLEASE:<br>
<br>
  *Stop sending Code-Review-1 when asking a question in a patch*<br>
<br>
_Sometimes_ there are good reasons to set -1 even when asking a<br>
question. For example, when the question is a hint sent to the patch<br>
author so that (s)he improves is commit message, a code comment or a<br>
piece of code.<br>
<br>
But most of the time, if you ask a question because there's something<br>
YOU DO NOT KNOW OR UNDERSTAND, do not put a score to a patchset. You<br>
don't know the answer, so you have absolutely no right to evaluate a<br>
patchset with -1. Just don't set a score, it's OK, and wait for the<br>
answer before deciding if the patch is worth [-1..+2].<br>
<br>
Thank you for listening, and happy hacking!<br>
<br>
-- <br>
Julien Danjou<br>
;; Free Software hacker<br>
;; <a href="http://julien.danjou.info">http://julien.danjou.info</a><br>
</div>
</span></font>
</body>
</html>