No objections from me!<br><br><div class="gmail_quote"><div dir="ltr">On Fri, 26 Oct 2018, 07:43 Tim and Alison Bentley, <<a href="mailto:Home@trarbentley.net">Home@trarbentley.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto">Seems reasonable to me.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, 26 Oct 2018, 05:43 Raoul Snyman via openlp-dev, <<a href="mailto:openlp-dev@openlp.io" target="_blank">openlp-dev@openlp.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2018-10-25 21:35, Tomas Groth wrote:<br>
> The test_pylint is not as old as you think and is actually responsible <br>
> for<br>
> linting on Jenkins on the old setup... Also it should only run if you<br>
> target that script specifically, so typically not on local setups. If <br>
> the<br>
> new CI lint solution is superior it is ok to remove it.<br>
<br>
The new merge proposal pipeline in Jenkins uses pycodestyle directly.<br>
<br>
The older branch linting uses the pylint test. I think we should move <br>
away from the PyLint test and do linting directly with the 3rd party <br>
tools like the new merge proposal pipeline does.<br>
<br>
-- <br>
Raoul Snyman<br>
<a href="mailto:raoul@snyman.info" rel="noreferrer" target="_blank">raoul@snyman.info</a><br>
_______________________________________________<br>
openlp-dev mailing list<br>
<a href="mailto:openlp-dev@openlp.io" rel="noreferrer" target="_blank">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>
_______________________________________________<br>
openlp-dev mailing list<br>
<a href="mailto:openlp-dev@openlp.io" target="_blank">openlp-dev@openlp.io</a><br>
<a href="https://lists.openlp.io/mailman/listinfo/openlp-dev" rel="noreferrer" target="_blank">https://lists.openlp.io/mailman/listinfo/openlp-dev</a><br>
</blockquote></div>