<div dir="ltr">One option is to run the test suite with go 1.3 and just make sure that juju compiles with all the other compilers (1.2, gccgo, etc). That gives us a fast precommit check, which won't catch everything, but should catch most 1.2 compat bugs. And leave the full test suite runs as CI tests.<div><br></div><div>John</div><div>=:-></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 5, 2015 at 6:28 AM, Andrew Wilkins <span dir="ltr"><<a href="mailto:andrew.wilkins@canonical.com" target="_blank">andrew.wilkins@canonical.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"><span class="">On Thu, Mar 5, 2015 at 10:24 AM, Tim Penhey <span dir="ltr"><<a href="mailto:tim.penhey@canonical.com" target="_blank">tim.penhey@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 04/03/15 19:05, Andrew Wilkins wrote:<br>
> The bugs were fixed in this branch: <a href="https://github.com/juju/juju/pull/1738" target="_blank">https://github.com/juju/juju/pull/1738</a><br>
</span><span>>  - invalid printf-style formatting will cause "go vet" to fail<br>
<br>
</span>My bad. I had deleted the pre-push hook script some time ago when go vet<br>
failed continuously, and I hadn't reinstated it.<br>
<br>
I have done so now.<br></blockquote><div><br></div></span><div>Thanks. FYI (and others' I), there's a couple of ways to temporarily disable hooks or just vet:</div><div><br></div><div>    git push --no-verify</div><div>or</div><div>    IGNORE_VET_WARNINGS=1 git push</div><div><br></div><div>(obviously to be used judiciously)</div></div></div></div>
<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>
<br></blockquote></div><br></div>