Contributors mailing list archives

contributors@odoo-community.org

Browse archives

Avatar

Pylint checks to solve or not

by
Akretion France, David BEAL
- 08/07/2020 17:09:35
Hi all,

I have a doubt about pylint checks that have to be solved, mandatory or not ?

Example with this file
https://travis-ci.org/github/OCA/edi/jobs/705389557

I have checked with C prefix with bold form sentence like here
https://travis-ci.org/github/OCA/edi/jobs/705389557#L306

I have others with W prefix (probably means warning)

If I assume only C checks need to be mandatory solved, is it exact ?

I'm a little surprised that check in PR
https://github.com/OCA/edi/pull/172
are different from checks in final merge.

see travis of last commit
https://github.com/OCA/edi/commits/8eb114944a4fcd9fc76b2a53e9078b742f7ee76f

But I can live with it if you I have an answer in my first question.

Thanks a lot

Bonne journée


David BEAL - akretion.com
Consultant
Odoo Intégration / Développement

Follow-Ups