<div class="gmail_quote">On Wed, May 2, 2012 at 2:17 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":33o">those patches have much better chances of being reviewed (at least by<br>
me) if you:<br>
- split them into one mail per patch</div></blockquote></div><br><div>Just to avoid confusion on this front, as I feel like I may have given contradictory advice here, let me briefly elaborate...</div><div><br></div><div>
Sending one mail with N patches *on a particular topic* is better than simultaneously sending N mails with N patches... However, if the contents of each patch are largely unrelated, that's no-longer the case. In this case, it seems like the last two patches are the only two closely related; i would keep those grouped in a single mail.</div>
<div><br></div><div>However, better still is to send each individual patch in its own email thread the moment it's ready rather than batching them up. This avoids hitting folks with a pile of reviews all at once and generally works better with the way the community reviews patches.</div>
<div><br></div><div>Anyways, I'll go back to the corner now, as I know nothing about Python. =]</div><div>-Chandler</div>