[openstack-dev] Please stop reviewing code while asking questions

Gorka Eguileor geguileo at redhat.com
Fri Apr 24 08:28:45 UTC 2015


On Fri, Apr 24, 2015 at 10:14:38AM +0200, Julien Danjou wrote:
> Hi there,
> 
> This is now happening weekly to me now, probably because I write too
> many patches touching almost all OpenStack projects once a cycle, and
> I'm really tired of that behavior, so PLEASE:
> 
>   *Stop sending Code-Review-1 when asking a question in a patch*
> 
> _Sometimes_ there are good reasons to set -1 even when asking a
> question. For example, when the question is a hint sent to the patch
> author so that (s)he improves is commit message, a code comment or a
> piece of code.
> 
> But most of the time, if you ask a question because there's something
> YOU DO NOT KNOW OR UNDERSTAND, do not put a score to a patchset. You
> don't know the answer, so you have absolutely no right to evaluate a
> patchset with -1. Just don't set a score, it's OK, and wait for the
> answer before deciding if the patch is worth [-1..+2].
> 
> Thank you for listening, and happy hacking!
> 
> -- 
> Julien Danjou
> ;; Free Software hacker
> ;; http://julien.danjou.info

+1

It does bother me too, especially when you answer the question and you
never hear back from them and the -1 stays there...  XD


Gorka



More information about the OpenStack-dev mailing list