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