<div dir="ltr">On Sat, Feb 2, 2013 at 5: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">On Sat, Feb 2, 2013 at 4:14 PM, 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">I think non-working is a misnomer here. These do not break the build.</blockquote></div><br>They issue warnings and the code can't possibly execute correctly. Just don't push it (or push it somewhere else) until it's been cleaned up to the point where it's not wasting our time to review.</div>
</div></blockquote><div><br></div><div style>I would love it if people never pushed code with any bugs. This is possible, just not efficient. All changes to</div><div style>workflow should be evaluated on this basis. For this particular change,</div>
<div style><br></div><div style>  1) It does not break the build</div><div style><br></div><div style>  2) Its a new feature, so does not break tests except the ones its supposed to</div><div style><br></div><div style>I do think warnings are annoying and people should endeavor to push code with no</div>
<div style>warnings, but making this a requirement takes away flexibility while providing nothing</div><div style>I can see except lack of Jed Annoyance. The claim that you are code reviewing this</div><div style>push is false on its face.</div>
<div style><br></div><div style>I think this play into a larger issue. Does the increased process burden you are recommending</div><div style>actually bring increased productivity. That the answer is yes is not at all clear. It is easy to make</div>
<div style>the mistake that because something sounds good and "right" that you should do it. This dooms</div><div style>most CS projects.</div><div style><br></div><div style>   Matt</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">Related: I would like to start tweaking our workflow to make petsc-dev more consistently stable, so that more applications can work with it instead of needing to wait for a release. Having people pushing code that doesn't work, isn't tested, and obviously wouldn't pass review is not good for stability.</div>

</div>
</blockquote></div><br><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>