<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Apr 18, 2015 at 9:30 PM, Boris Pavlovic <span dir="ltr"><<a href="mailto:boris@pavlovic.me" target="_blank">boris@pavlovic.me</a>></span> wrote:<br><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"><div>Code <span class="">coverage</span> is one of the very important metric of overall code quality especially in case of Python. It's quite important to ensure that code is <span class="">covered</span> fully with well written unit tests. </div><div><br></div><div>One of the nice thing is <span class="">coverage</span> job. </div></blockquote></div><br>I really like the idea of adding the coverage job everywhere so that developers can view the results be using a link in Gerrit. I think this would make it easier for many.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I don't like the idea of checking that coverage is increased. There are many issues with that. The two biggest one for me are:</div><div class="gmail_extra"><br></div><div class="gmail_extra">1. It will either lead to people doing things to game the system or overuse of the #<span style="font-size:12.8000001907349px">no-</span><span class="" style="font-size:12.8000001907349px">coverage</span><span style="font-size:12.8000001907349px">-check  tag you mentioned.</span><br></div><div class="gmail_extra"><span style="font-size:12.8000001907349px"><br></span></div><div class="gmail_extra"><span style="font-size:12.8000001907349px">2. It really doesn't tell you too much. A core developer should really be looking at the tested use cases to see if they are all there. Line coverage and even branch coverage won't tell you that. <br></span><br clear="all"><div><br></div>-- <br><div class="gmail_signature">David<br>blog: <a href="http://www.traceback.org" target="_blank">http://www.traceback.org</a><br>twitter: <a href="http://twitter.com/dstanek" target="_blank">http://twitter.com/dstanek</a><div>www: <a href="http://dstanek.com" target="_blank">http://dstanek.com</a></div></div>
</div></div>