Contributors mailing list archives
contributors@odoo-community.org
Browse archives
Pylint checks to solve or not
by
Akretion France, David BEAL
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
-
-
Re: Pylint checks to solve or not
byOpen for Small Business, Graeme Gellatly