On Oct 19, 2010, at 02:15 , Robert Collins wrote:

> On Tue, Oct 19, 2010 at 5:42 AM, Henning Eggers
> 
> 
>> Please talk to Brad about both issues and see what he says.
> 
> I'm not sure why Brad specifically - the review team was always a
> democracy ever since it was formulated : we really want broad input,
> still we can start with Brad... Brad, what do you think?

The review team is, of course, a democracy.  I think Henning was probably just 
looking for a sanity check before involving the entire team.

As Rob indicated, the last few AsiaPac reviewers meetings have spent 
considerable time talking about lots of wide-ranging topics regarding reviews.  
Some of those ideas have been written up now in a subsequent email Rob sent 
proposing the optional review experiment.

A discussion of eliminating tech debt was brought up by Curtis recently in the 
AmEu meeting ("don't add to canonical.launchpad as we're trying to get rid of 
it") and was uncontroversial.  We all agree tech debt is a burden for many 
reasons and in a perfect world we would make it go away.  Curtis proposed one 
approach and most people thought it was reasonable.  Robert has a different 
view, as expressed here, and in the AsiaPac meetings.

The topic comes up in the reviewers meeting because a) it is the only meeting 
where we all get together regularly, ignoring the fact we are forced to do it 
regionally with a single conduit (me), and b) the review is the last chance for 
a peer to catch the fact you may be contributing to tech debt or simply not 
cleaning it up.

The "you touch it, you fix it" mode of working is certainly not fair but it 
does cut across the board if we all agree to the idea.  I have no qualms as a 
reviewer saying "I know you didn't create this problem but while you're there 
would you spend a few minutes to clean it up?"  Most people are happy to 
oblige, not because I am wielding enormous reviewer power, but because we all 
agree we want to get rid of warts.

As to the fear that people will be unwilling to wander into the dark corners of 
the code base for fear they will be accountable for cleaning it all up, I'm not 
convinced that will be the way most of us will work.  Our enforcement of 
cleaning up things you touch is not heavy handed.  Henning asked that you have 
a follow-on branch but did not demand it as a condition of approval.  It is 
simply our agreement that we'll all share, more or less, in the less 
interesting chore of cleaning up after ourselves.

Robert I think your views expressed here are bigger than the issue at hand and 
represent an idea that our development process, not just our review process, is 
fundamentally flawed.  I think you need to find the appropriate forum to make 
suggestions and solicit input from everyone.
-- 
https://code.launchpad.net/~lifeless/launchpad/uniqueconfig/+merge/38689
Your team Launchpad code reviewers is requested to review the proposed merge of 
lp:~lifeless/launchpad/uniqueconfig into lp:launchpad/devel.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : launchpad-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to