<p>To also give my input on this - I'm all for reviewing the code before merging, especially for bigger patches. I even try to read all reviews myself, because it's anothet way how to learn writting better code. But on the other hand, for small (obvious) patches I feel it's uselessly slowing things down. But then again, some might learn even from these small reviews. So I don't know really. I think I'll get used to whatever we'll decide here. <br>
For now, let's focus on merging my clone repo back to the master tree. I'll update the review diff when I'll get home later tonight, so let's start reviewing it then.</p>
<p>Marty</p>
<div class="gmail_quote">On Mar 23, 2011 3:25 PM, "David Edmundson" <<a href="mailto:david@davidedmundson.co.uk">david@davidedmundson.co.uk</a>> wrote:<br type="attribution">> This conversation came up in IRC and I wanted to bring it to the mailing<br>
> list:<br>> <br>> - I want to merge Martin's contact list into the main repository.<br>> -Martin is worried that if he has to start going through reviewboard for<br>> patches it's really going to slow down development.<br>
> <br>> I can see where he's coming from, when I see one line patches on reviewboard<br>> it wastes my time going through sorting them out, and it does slow things<br>> down. Especially when everyone hacking is doing this just in the few moments<br>
> of their freetime.<br>> <br>> On the other hand I have fixed a lot of code in reviewboard before it's<br>> merged, and had a lot of my mistakes spotted too.<br>> <br>> Personally I just want his stuff merged, and I'm happy to leave the choice<br>
> of whether to enforce reviewboard or not to the discretion of the component<br>> leader.<br>> <br>> Does anyone have any thoughts?<br>> <br>> Dave<br></div>