<div dir="ltr">The bot cannot make changes to the tree and commit it back to trunk, because it doesn't have direct push rights to github. Instead it has API rights to "accept this merge". So even if it did run the changes, it wouldn't end up in trunk, and we could only reject the merge if it changed things. (As I understand it)<div><br><div>John</div><div>=:-></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 24, 2015 at 3:56 PM, David Cheney <span dir="ltr"><<a href="mailto:david.cheney@canonical.com" target="_blank">david.cheney@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Why doesn't the bot run this hook ?<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Apr 24, 2015 at 9:54 PM, Matthew Williams<br>
<<a href="mailto:matthew.williams@canonical.com">matthew.williams@canonical.com</a>> wrote:<br>
> There's a very useful pre-push hook available for juju that does useful<br>
> things like run gofmt, go vet, checks we can build etc.<br>
><br>
> If you don't have it setup there's some lovely instructions in<br>
> Contributing.md:<br>
><br>
> <a href="https://github.com/juju/juju/blob/master/CONTRIBUTING.md#local-clone" target="_blank">https://github.com/juju/juju/blob/master/CONTRIBUTING.md#local-clone</a><br>
><br>
> Thanks for listening<br>
><br>
> Matty<br>
><br>
</div></div><span class="HOEnZb"><font color="#888888">> --<br>
> Juju-dev mailing list<br>
> <a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
> Modify settings or unsubscribe at:<br>
> <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
><br>
<br>
--<br>
Juju-dev mailing list<br>
<a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
</font></span></blockquote></div><br></div>