No objections from me!

On Fri, 26 Oct 2018, 07:43 Tim and Alison Bentley, <[email protected]>
wrote:

> Seems reasonable to me.
>
> On Fri, 26 Oct 2018, 05:43 Raoul Snyman via openlp-dev, <
> [email protected]> 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
>> [email protected]
>> _______________________________________________
>> openlp-dev mailing list
>> [email protected]
>> https://lists.openlp.io/mailman/listinfo/openlp-dev
>>
> _______________________________________________
> openlp-dev mailing list
> [email protected]
> https://lists.openlp.io/mailman/listinfo/openlp-dev
>
_______________________________________________
openlp-dev mailing list
[email protected]
https://lists.openlp.io/mailman/listinfo/openlp-dev

Reply via email to