[packagekit] pylint madness

[Tim Lauridsen]

Maybe we should have an discussion about the use of pylint:

- (name, idver, a, repo) = self.get_package_from_id(package_id)

+ (name, _, _, repo) = self.get_package_from_id(package_id)


A change like this, to silence pylint is bad IMHO. it is better to 
disable the pylint check for unused vars.
then to make this weird stuff, it really make the code harder to read IMHO.

pyint is make made to check the code for the coding standards done by 
the firm who has made pylint, not
for the upstream python coding standard, so a lot of checks has to be 
disabled because they are totally insane.

pylint is good to find error in then code, but a lot of the coding 
standard stuff need to be turned off.


Tim

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/packagekit/attachments/20080928/48bd5561/attachment-0003.htm>