Ryan Harper has posted comments on this change. Change subject: pylint:vdsm/*.py cast output to str() for .strip()/splitlines() ......................................................................
Patch Set 2: I tend to agree with pylint bug. It's seems rather annoying to have to always do the str() casting. And it clear from runtime this works out fine. The other area of pylint "errors" stem from the same inability to detect the runtime association of types. The blog post talked about how to teach pylint more about your code; but that seemed like a lot of work for little benefit. -- To view, visit http://gerrit.ovirt.org/3444 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iae2c0ece02d5e41e0080bf41c0d9ea3544965274 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ryan Harper <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ryan Harper <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
