<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 20, 2013 at 8:48 PM, Chris Cormack <span dir="ltr"><<a href="mailto:chris@bigballofwax.co.nz" target="_blank">chris@bigballofwax.co.nz</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">One question, would you like the bot to comment if it applied a patch<br>
cleanly? Or just move on to the next one, (as it is doing now).<br>
The next thing I plan to add, is having it run the test suite, or at<br>
least the 00- tests. Would people like it to mark things failed qa if<br>
the tests fail, or just leave a comment?<br></blockquote></div><br clear="all"><div>I think it should make a comment if it successfully applies the patch (or maybe set a custom last-looked-at-by-bot field?).  I also think that it should set the bug to failed QA if whatever tests are run don't pass, but I imagine some tweaking will be required so that it doesn't do things like auto-fail a patch that adds a new dependency -- so maybe it should just add a comment at first.  Once we're confident that it isn't generating false negatives, then we could have it change the status.</div>
<div><br></div><div>Will the bot process bugs again after a suitable interval has passed?  A patch that applies today may not apply two months from now if master has moved on but no human has yet looked at the bug.</div><div>
<br></div><div>Regards,</div><div><br>Galen</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>