<div dir="ltr">Hi,<div><br></div><div>Recently we have introduced a hard check of 100% coverage for all changes.</div><div>This is done via coverage + codecov + github protected branches.</div><div><br></div><div>Now, if your patch is not 100% covered github will not let you merge it.<br clear="all"><div><br></div><div>See for example this change: <a href="https://github.com/twisted/twisted/pull/261/files#diff-0fea8a8ca713deb7ea6a10053273319aR2360">https://github.com/twisted/twisted/pull/261/files#diff-0fea8a8ca713deb7ea6a10053273319aR2360</a></div><div><br></div><div>The errback is there to help with test failures ... but the test should never fail, so that errback is never called... and that line is not covered.</div><div><br></div><div>How should we proceed with these changes?</div><div><br></div><div>Maybe this is not the best example and that code could be refactored... but I think that the topic of ignoring missing coverage is still valid.</div><div><br></div><div>I suggest to introduce `  # pragma: no cover`</div><div><br></div><div>and update the coverage config with</div><div><br></div><div><div>[report]</div><div>exclude_lines =</div><div>    pragma: no cover</div></div><div><br></div><div>-----------</div><div><br></div><div>What do you think?</div><div><br></div><div>Thanks!</div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Adi Roiban</div>

</div></div>