[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