[packagekit] pylint madness

Anders F Björklund afb at algonet.se
Sun Sep 28 00:47:59 PDT 2008


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.

I just "fixed" the whitespace warnings, and ignored the rest of them,
when testing running smartBackend through the automated complainer...

Isn't there some kind of pylint configuration that could be included ?
But I still think that using spaces in the method calls is a good idea.

--anders




More information about the PackageKit mailing list