[openstack-dev] Please stop reviewing code while asking questions
joe.gordon0 at gmail.com
Fri Apr 24 18:26:20 UTC 2015
On Fri, Apr 24, 2015 at 11:16 AM, Julien Danjou <julien at danjou.info> wrote:
> On Fri, Apr 24 2015, Joe Gordon wrote:
> > When I get a -1 on one of my patches with a question, I personally treat
> > as a short coming of the commit message. To often in the past I have
> > at a file, and in trying to figure out why that line is there I do a git
> > blame only to see a useless commit message with me as the author.
> That's a thing that I've been stated over and over again in this thread
> and actually paraphrased from the first paragraph on my original email.
> It'd be cool if we could stop restating the obvious over and over again.
So you did, sorry.
> Could someone give me an example of how we are supposed to improve the
> patch or commit message when one get a -1 with e.g. the question:
> "Why do you use getattr(foo, "bar", None)?"
By calling them out in the review or on irc, and explain to them when its
appropriate to use a -1. I don't think its safe to assume that a
significant number of people who do these -1s are read every thread on the
> when the answer is "Well, otherwise it will raise an error and the code
> will fail" because the reviewer do not know how getattr() works.
> Julien Danjou
> // Free Software hacker
> // http://julien.danjou.info
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the OpenStack-dev