<div dir="ltr">On Sun, Feb 3, 2013 at 12:20 PM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov" target="_blank">jedbrown@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><div class="im">On Sun, Feb 3, 2013 at 11:16 AM, Matthew Knepley <span dir="ltr"><<a href="mailto:knepley@gmail.com" target="_blank">knepley@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra"><div class="gmail_quote"><div>I did address. It would be great if people never pushed warnings. I try not to.</div>

</div></div></blockquote><div><br></div></div><div>You push new warnings almost every day.</div></div></div></div></blockquote><div><br></div><div style>Hyperbole does not serve rational argument.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><span style="color:rgb(80,0,80)"> </span></div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_extra">
<div class="gmail_quote"><div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Pushing as a checkpointing mechanism discourages review.</div></div></div></div></blockquote><div><br></div></div><div>Review should happend when the section is complete, but this is no way implies that you should not</div>


<div>push until it is complete.</div></div></div></blockquote><div><br></div></div><div>How do you identify what the feature is when it's in 10 commits interspersed over 200 in the history. My claim is that you should make those 10 commits on top of each other without merging (unless you need something specific that was pushed to petsc-dev) and merge when it's complete. Pushing to petsc-dev should _mean_ that it's ready for review. This does not take more work.</div>

</div></div></div>
</blockquote></div><br>Again, hyperbole is not useful. This is a single commit, where I add functionality to a few functions for a single purpose. Are</div><div class="gmail_extra">you even reading this before commenting?</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">  Matt<br clear="all"><div><br></div>-- <br>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
-- Norbert Wiener
</div></div>