Are you talking about pushing your commits 26d55f7a59 thru bb4b221faf2?  My opinion is that these commits are not ready to be pushed, but I have not had a lot of time to do a proper review.  For ease of reviewing, I'd recommend squashing all your commits into two- the first commit performs adds your new feature, the second commit that adds the tests.  Then use git format-patch and attach the commits to the mailing list.<br>
<br>Pat<br><br><br><br><div class="gmail_quote">On Wed, Sep 22, 2010 at 8:14 AM, David Doria <span dir="ltr"><<a href="mailto:daviddoria@gmail.com">daviddoria@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">> Did anyone get a chance to look at this?<br>
> David<br>
<br>
</div>Can I push this?<br>
<font color="#888888"><br>
David<br>
</font></blockquote></div><br>