<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Oct 2, 2014 at 1:35 PM, Martin Packman <span dir="ltr"><<a href="mailto:martin.packman@canonical.com" target="_blank">martin.packman@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The landing job for juju now uses the pre-push script from the juju<br>
branch before running tests. This means it's running go vet over the<br>
source, and I've fixed a couple of errors on master.<br></blockquote><div><br></div><div>Great, thanks Martin.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What it doesn't do currently is fail the landing if vet reports<br>
issues, as the script doesn't fail in this case. If we want to make go<br>
vet issues gate the landing of code, we can just land that change to<br>
the pre-push script.</blockquote><div><br></div><div>IIRC, "go tool vet" will only set an error return code when invoked on a package or files, rather than on a directory. I think it's a matter of checking whether "go tool vet" prints any warnings, rather than relying on the return code. Or xargs and get the RC.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
Martin<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></div>