[teampractices] Code review before writing new code

James Douglas jdouglas at wikimedia.org
Tue Jun 9 16:44:44 UTC 2015


> I think Arthur's comment was based on the theory that a lot of the code
to be reviewed by a team may not be related to work that is not that team's
priority.

Wouldn't that a misprioritization?


On Tue, Jun 9, 2015 at 9:35 AM, Kevin Smith <ksmith at wikimedia.org> wrote:

> On Tue, Jun 9, 2015 at 9:25 AM, James Douglas <jdouglas at wikimedia.org>
> wrote:
>
>> > staying focussed on their teams' priorities may in fact contribute to
>> code review queues growing
>>
>> I disagree, unless the team's priority is "add to the code review queue".
>> :]
>>
>
> I think Arthur's comment was based on the theory that a lot of the code to
> be reviewed by a team may not be related to work that is not that team's
> priority. There was a lot of that in the old core team, where they had to
> review code in a variety of areas, and doing so slowed their coding
> progress on work directly related to their quarterly goal. If they stayed
> purely focused on their own priorities, code reviews (of code written by
> people outside their team) would pile up.
>
> Kevin
>
>
> _______________________________________________
> teampractices mailing list
> teampractices at lists.wikimedia.org
> https://lists.wikimedia.org/mailman/listinfo/teampractices
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.wikimedia.org/pipermail/teampractices/attachments/20150609/58194dd9/attachment.html>


More information about the teampractices mailing list