Giuseppe Vallarelli has posted comments on this change. Change subject: lint: providing a default pylintrc ......................................................................
Patch Set 4: Hi Alon, I think you might need to customize the pylint for the specific project. Clearly there might be some false positives but for example it can detect not consistent naming conventions throughout the project as well as some errors like unused variables, no exceptions type speficied, builtins that get redefined (a few errors that I've found running pylint). -- To view, visit http://gerrit.ovirt.org/17772 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia41d2bf225e0b5e47a0e2718b906fbd44f5a54e9 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Giuseppe Vallarelli <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Giuseppe Vallarelli <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
