<div dir="auto"><div>Hi,</div><div dir="auto"><br></div><div dir="auto">The test_pylint is not as old as you think and is actually responsible for linting on Jenkins on the old setup... Also it should only run if you target that script specifically, so typically not on local setups. If the new CI lint solution is superior it is ok to remove it.</div><div dir="auto"><br></div><div dir="auto">The test_bzr_tags I don't know, but does not seem to be that important.</div><div dir="auto"><br></div><div dir="auto">/Tomas<br><br><div class="gmail_quote" dir="auto"><div dir="ltr">fre. 26. okt. 2018 04.32 skrev Raoul Snyman via openlp-dev <<a href="mailto:openlp-dev@openlp.io">openlp-dev@openlp.io</a>>:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi folks,<br>
<br>
I'd like to remove tests/utils/test_pylint.py and <br>
tests/utils/test_bzr_tags.py. They are currently just making running <br>
tests and code coverage a nightmare, and haven't served a purpose in <br>
years.<br>
<br>
We have done linting in Jenkins for years now, so I don't think the <br>
linting tests are necessary anymore.<br>
<br>
Additionally, the bzr tags test was to prevent a future tag mess that we <br>
haven't come across since the previous tag mess which was years and <br>
years ago, and I don't think it is necessary any longer.<br>
<br>
Yea/Nay?<br>
<br>
-- <br>
Raoul Snyman<br>
+1 (520) 490-9743<br>
<a href="mailto:raoul@snyman.info" target="_blank" rel="noreferrer">raoul@snyman.info</a><br>
_______________________________________________<br>
openlp-dev mailing list<br>
<a href="mailto:openlp-dev@openlp.io" target="_blank" rel="noreferrer">openlp-dev@openlp.io</a><br>
<a href="https://lists.openlp.io/mailman/listinfo/openlp-dev" rel="noreferrer noreferrer" target="_blank">https://lists.openlp.io/mailman/listinfo/openlp-dev</a><br>
</blockquote></div></div></div>