[openlp-dev] Removing test_pylint.py and test_bzr_tags.py

[Philip Ridout]

No objections from me!

On Fri, 26 Oct 2018, 07:43 Tim and Alison Bentley, <Home at trarbentley.net>
wrote:

> Seems reasonable to me.
>
> On Fri, 26 Oct 2018, 05:43 Raoul Snyman via openlp-dev, <
> openlp-dev at openlp.io> wrote:
>
>> On 2018-10-25 21:35, Tomas Groth wrote:
>> > 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.
>>
>> The new merge proposal pipeline in Jenkins uses pycodestyle directly.
>>
>> The older branch linting uses the pylint test. I think we should move
>> away from the PyLint test and do linting directly with the 3rd party
>> tools like the new merge proposal pipeline does.
>>
>> --
>> Raoul Snyman
>> raoul at snyman.info
>> _______________________________________________
>> openlp-dev mailing list
>> openlp-dev at openlp.io
>> https://lists.openlp.io/mailman/listinfo/openlp-dev
>>
> _______________________________________________
> openlp-dev mailing list
> openlp-dev at openlp.io
> https://lists.openlp.io/mailman/listinfo/openlp-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openlp.io/pipermail/openlp-dev/attachments/20181026/450d11cf/attachment.html>