<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 11, 2013 at 9:37 AM, Jared Camins-Esakov <span dir="ltr"><<a href="mailto:jcamins@cpbibliography.com" target="_blank">jcamins@cpbibliography.com</a>></span> wrote:<br>
<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>Signing off is both checking that a feature works and -- hopefully -- considering whether there are any bigger problems with it, such as regressions, to the extent that the developer is able to do this. Unit tests provide an easy way to test for regressions, which is why it's a good idea to run them -- if not all of them, at least the relevant ones -- while signing off. Obviously as RM I do not have the power to mandate that everyone must run unit tests before signing off, nor would I want to, but I will say that the more thorough the sign off, the more likely it is that I will spend time on a patch. Patches with only a perfunctory sign off are more likely to trigger my "one strike" rule (that's the rule that says the first problem I encounter with a patch, no matter how minor, can get the patch failed).</div>

<div><br></div><div>The QA team should also be running unit tests, of course, but the point of the sign off/QA divide is that we want more eyes reviewing patches before they go in. If the developer who signs off is running unit tests, I can feel pretty confident that they didn't just add a "Signed-off-by" line before uploading a patch.</div>
</div></div></div></blockquote><div><br></div><div style>I cannot give this enough +1s.  QA *must* be considered the responsibility of everybody who regularly submits patches to Koha, not just the QA team.  I consider making sure that the tests pass if you submit a patch or if you sign off on one to be the *bare minimum* of what we should expect of ourselves.</div>
<div style><br></div><div style>Regards,</div><div style><br></div><div style>Galen </div></div>-- <br><div dir="ltr"><div>Galen Charlton</div><div>Manager of Implementation</div><div>Equinox Software, Inc. / The Open Source Experts</div>
<div>email:  <a href="mailto:gmc@esilibrary.com" target="_blank">gmc@esilibrary.com</a></div><div>direct: +1 770-709-5581</div><div>cell:   +1 404-984-4366</div><div>skype:  gmcharlt</div><div>web:    <a href="http://www.esilibrary.com/" target="_blank">http://www.esilibrary.com/</a></div>
<div>Supporting Koha and Evergreen: <a href="http://koha-community.org" target="_blank">http://koha-community.org</a> & <a href="http://evergreen-ils.org" target="_blank">http://evergreen-ils.org</a></div></div>
</div></div>