<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 6, 2016 at 7:08 PM, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, 6 Jun 2016, Matthew Knepley wrote:<br>
<br>
</span><span class="">> On Mon, Jun 6, 2016 at 5:27 PM, Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> wrote:<br>
><br>
> > Matt,<br>
> ><br>
> > Per integration workflow - all feature branches should be tested<br>
> > locally (and be complete) - before merged to next [next is for<br>
> > integration testing - not feature testing]<br>
> ><br>
> > You could have used (modified):<br>
> ><br>
> > config/examples/arch-linux-xsdk-dbg.py<br>
> > config/examples/arch-osx-xsdk-opt.py<br>
> ><br>
> > Yeah - we don't have automatic 'feature branch test before integration<br>
> > testing' workflow - so currently this has to be done manually.<br>
><br>
><br>
> I don't really have the OS/compiler options available to test things<br>
> exhaustively, so I am<br>
> using next for this. I think this is acceptable right now.<br>
<br>
</span>Well then the feature is not yet ready for next - and could have<br>
waited until it was ready.<br></blockquote><div><br></div><div>I do nto agree here. Just because some Git nerd thinks that 'next' should be</div><div>that way does not mean it is what makes us most productive. I think we are way</div><div>more productive using the nightly tests as a way to discover bugs. I cannot waste</div><div>my personal time running a bunch of tests on my own slow laptop before pushing.</div><div><br></div><div>  Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If it was for someone to use - then they could have used the<br>
feature-branch..<br>
<br>
The workflow requires the feature to be complete - and minimally<br>
tested - before merge to next.. We do ocassionally take shortcuts -<br>
but this sometimes this results in extended broken next...<br>
<span class="HOEnZb"><font color="#888888"><br>
Satish<br>
<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">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></div>