<div><font color="#000000"><font>I've gotten a lot of pull requests over the past month, and I've been reviewing them today.<br clear="all"></font></font></div><div><font color="#000000"><font><br></font></font></div>

<div><font color="#000000"><font>I'm seeing two main issues:</font></font><font color="#000000"><font><br></font></font></div><div><font color="#000000"><font><br></font></font></div><div><font color="#000000"><font>1) <strong>Nobody's writing any tests.</strong>  We have 10 recent pull requests, and not a test amongst them.  Tests prove that a problem existed under the old code, and that it's fixed with the new code.  Also, it demonstrates what your code does.  We consider tests to be very valuable, and are very hesitant to accept patches without them.  There are many existing tests in UnitTests to look at if you are not familiar with NUnit.  If you feel that tests are unwarranted, please state your case in a Pull Request comment.</font></font><font color="#000000"><font><br>

</font></font></div><div><font color="#000000"><font><br></font></font></div><div><font color="#000000"><font>2) <strong>We can't break backward compatibility.  </strong></font></font><font color="#000000"><font> A couple patches are removing or altering public functions.  We can't do that.  Anybody who upgrades their QF/n release should expect that it should compile with their existing app just fine.</font></font><font color="#000000"><font><br>

</font></font></div><div><font color="#000000"><font><br></font></font></div><div><font color="#000000"><font>If you propose that a function should be removed, mark it with </font></font>[System.Obsolete("what to do instead")]<font color="#000000"><font> (so the compiler gives a warning), but you must leave it in.<br>

</font></font></div><div><font color="#000000"><font><br></font></font></div><div><font color="#000000"><font>Thanks.<br></font></font></div><div><font color="#000000"><font>-Grant<br></font></font></div><div><font color="#000000"><font><br>

</font></font></div><br>-- <br><div><font><span style="background-color:#000000"><span style="background-color:#ffffff">Grant Birchmeier</span></span></font><font><font color="#3333ff"><br></font></font></div><div><font><b><font color="#3333ff"><span style="background-color:#ffcc33"><span style="background-color:#ffcc66"><span style="background-color:#ffcc99"><span style="background-color:#ffffff"><span style="background-color:#ffcc00">Connamara Systems, LLC</span></span></span></span></span></font></b></font><br>

</div><div><font><b>Made-To-Measure Trading Solutions.</b></font></div><div><font>Exactly what you need. No more. No less.</font><font><b><font color="#3333ff"><br></font></b></font></div><div><font><a href="http://connamara.com" target="_blank">http://connamara.com</a><br>

</font></div><br>