[teampractices] Code review social norms
Rob Lanphier
robla at wikimedia.org
Wed Mar 16 06:33:26 UTC 2016
On Tue, Mar 15, 2016 at 8:03 PM, Mukunda Modell <mmodell at wikimedia.org>
wrote:
> On Tue, Mar 15, 2016 at 6:48 PM, Kevin Smith <ksmith at wikimedia.org> wrote:
>
>> I would mention that in some cases, I would prefer to accept the commit
>> as is, and then perform minor refactoring, such as changing a name, fixing
>> a typo, or rearranging the code. Not only does that clearly separate
>> authorship, but it would also encourage those changes to be reviewed by
>> someone other than that author.
>>
>
> This ^
>
> I think this says what I've been trying to say, only better.
>
Thank you Kevin and Mukunda. I think I still probably disagree with you,
but I understand what you're trying to say a lot better now, and I'm now in
the "mild disagreement" category.
My mild disagreement: I think it's good to have a system where people
collaborate on a patch before it lands in trunk/mainline. Subbu's case
seems reasonable to me.
Rob
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.wikimedia.org/pipermail/teampractices/attachments/20160315/efbced7c/attachment.html>
More information about the teampractices
mailing list