On Thu, May 20, 2010 at 5:42 PM, Adi Roiban <[email protected]> wrote: > Hi, > > In LP we have the following code: > > �...@operation_parameters( > pocket=Choice( > title=_("Pocket"), required=True, > vocabulary=DBEnumeratedType)) > > and it looks fine to me, but pylint complains about: > > lib/lp/registry/interfaces/sourcepackage.py > 191: [C0322, ISourcePackage.getBranch] Operator not preceded by a > space > required=True, > ^ > vocabulary=DBEnumeratedType)) > > > Danilo suggest to raise this issue on the mailinglist and see if we > should fix lint or our styleguide. >
No spaces around '=' for keyword arguments ever. World without end. > What do you think we should do to fix such warning from being raised by > `make lint` ? > Let's drop pylint as part of our "make lint" check. jml _______________________________________________ Mailing list: https://launchpad.net/~launchpad-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-dev More help : https://help.launchpad.net/ListHelp
