[openstack-dev] Please stop reviewing code while asking questions
Ihar Hrachyshka
ihrachys at redhat.com
Fri Apr 24 10:10:05 UTC 2015
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256
On 04/24/2015 10:14 AM, 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!
>
Generally +1. Neither reviewers should put -1 due to nitty nits for
typos in comments and especially in commit messages. There is a huge
price of it (we often note cpu cycles in this regard, but the actual
price is frustration and excessive attention to irrelevant "issues"
from fellow contributors). [Sometimes typos in comments and commit
messages really deserve -1, f.e. when they are not intelligible due to
those typos, but it's "one in a million" case.]
- From the other side, the problem with no score comments is that some
contributors assume those comments are not worth attention, since they
presumably don't influence the merge process.
And (almost) all comments are worth being answered. If not for the
particular patchset, if not for the author of the patch, then at least
for the reviewer.
Ihar
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2
iQEcBAEBCAAGBQJVOhZ9AAoJEC5aWaUY1u57FaoIAMg1cY7Rkif7SRBJdokBQ9Ky
R8tSkOPGripNJGxBc7NCfZOTeGFxU6cNI2pSpcEyZ3Gt256al87bWasbu8Drak2k
UEU0KsolOwjEa6z9uZ29Q0b3I2bGAIKk17otgsHC4eZ3jjdghO4IFgj1bVXR4kV1
dFX0vJtfaSGF3WEol2B1ZOtyi45fshM2inT/tr0S4qHxuejEvL+QN4IOhRmRcf6Z
w2BwtS2khIwj8HOHtsbd9KvQ5UqM8Vv4GQ5+Jz+864TRjZKiFWZFqYt47xyh9/Si
ZDXgZID0AdHv3mKtYOxpxhG5WtMUld4C7Ljm9sGKMgdCfLkevVyb3MxzoiEESJ0=
=/YE0
-----END PGP SIGNATURE-----
More information about the OpenStack-dev
mailing list