[Reviewers] Review Tooling

[Pierre-Yves David]

On 12/28/2016 05:35 PM, Augie Fackler wrote:
>
>> On Dec 28, 2016, at 11:34 AM, Pierre-Yves David <pierre-yves.david at ens-lyon.org> wrote:
>>
>>> Given how our other discussions have gone I wasn’t going to bring it up since it’s a policy change, and I’m weary of having policy discussions already. That said, the review tooling is cleaner than it used to be (Kevin and I paired on it for a day), and it should be feasible to add some testing and then add that as a feature. Should we call for a vote on the SC list, and plan to offer those permissions to those two as soon as the stack would support it?
>>
>> I think it is two early for these two, (but that they are on a way to be ready for it.)
>
> Do you object to me putting the question to a vote of the steering committee?

Yes, I do object. With the current state of the tooling, it is unclear 
the proposed changed will even help the patch flow and we have better 
things to consume our energy on.

If you want to improve the patch flow I suggest spending time on various 
things already discussed:

- improve drop flow (and reordering) for pending changesets),
- improve statistic gathering and display on the current flow,
- improve existing pre-reviewed flow.

Cheers,

-- 
Pierre-Yves David