<div dir="ltr">I think it's a great plan, I've already been caught with one minor rebase requirement and was glad to be prompted by the bot instead of wasting a humans time ;)<div><br></div><div>Martin</div></div><div class="gmail_extra">

<br clear="all"><div><div dir="ltr">Martin Renvoize<div>Software Engineer, PTFS Europe Ltd</div><div>Content Management and Library Solutions</div><div>Skype: <img src="http://mystatus.skype.com/bigclassic/ashimema" width="96" height="23"></div>

<div>Landline: 0203 286 8685</div><div>Mobile: 07725985636</div><div><br></div><div><a href="http://www.ptfs-europe.com" target="_blank">http://www.ptfs-europe.com</a></div></div></div>
<br><br><div class="gmail_quote">On 18 September 2013 15:31, Galen Charlton <span dir="ltr"><<a href="mailto:gmc@esilibrary.com" target="_blank">gmc@esilibrary.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">Hi,<br><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Wed, Sep 18, 2013 at 4:37 AM, Chris Cormack <span dir="ltr"><<a href="mailto:chris@bigballofwax.co.nz" target="_blank">chris@bigballofwax.co.nz</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">



You may have noticed that there have been a bunch of updates from<br>
<<a href="mailto:gitbot@bugs.koha-community.org" target="_blank">gitbot@bugs.koha-community.org</a>> to bugs in 'Needs Signoff' status.<br>
It is running through all bugs in that status, grabbing the patches<br>
and applying them, and changing the status if they don't apply clean.<br></blockquote><div><br></div></div><div>I think this is an excellent idea.</div><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">



If people find this annoying I will turn it off. If people don't I<br>
will start on phase 2, which is making it run all the tests after<br>
applying the patches, and catch any fails.<br></blockquote><div><br></div></div><div>I think the annoyance factor would mainly depend on how frequently the bot fails patches incorrectly.  I also have a concern about the potential for a new contributor to post a patch in good faith, only to have their first interaction with the community be a bot them that their patch is problematic, but I think we can wait and see how it works out in practice.</div>

<div class="im">
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
My idea is that everything in the 'Needs Signoff' should be passing<br>
tests and applying cleanly.<br></blockquote></div></div><div><br></div><div>Agreed.  One caveat, though -- if it isn't doing so already, the bot should be taking dependent bugs into account when testing whether patches apply.  It failing to do so would quickly turn it into an annoyance.<br>


</div><div><br></div><div>Regards,</div><div><br></div><div>Galen</div><span class="HOEnZb"><font color="#888888">-- <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: <a href="tel:%2B1%20770-709-5581" value="+17707095581" target="_blank">+1 770-709-5581</a></div><div>cell:   <a href="tel:%2B1%20404-984-4366" value="+14049844366" target="_blank">+1 404-984-4366</a></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>
</font></span></div></div>
<br>_______________________________________________<br>
Koha-devel mailing list<br>
<a href="mailto:Koha-devel@lists.koha-community.org">Koha-devel@lists.koha-community.org</a><br>
<a href="http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel" target="_blank">http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel</a><br>
website : <a href="http://www.koha-community.org/" target="_blank">http://www.koha-community.org/</a><br>
git : <a href="http://git.koha-community.org/" target="_blank">http://git.koha-community.org/</a><br>
bugs : <a href="http://bugs.koha-community.org/" target="_blank">http://bugs.koha-community.org/</a><br></blockquote></div><br></div>