Re: Changes to make pylint happy (continued)

2008-07-11 Thread Terry Brown
On Wed, 2 Jul 2008 08:59:01 -0700 (PDT) Edward K. Ream [EMAIL PROTECTED] wrote: So I'm lazy, but it's up to plugin authors to do real testing. In particular, unit tests will be far superior to any pylint tests, but I haven't been successful in convincing authors of plugins to create such

Re: Changes to make pylint happy

2008-07-02 Thread Edward K. Ream
On Tue, Jul 1, 2008 at 8:59 PM, Terry Brown [EMAIL PROTECTED] wrote: I've fixed some errors in cleo, graphed, and UNL (I'm not UNL's author, but I wrote the code it was complaining about). These errors remain: * Module leo.plugins.UNL W0102:129:recursiveUNLSearch: Dangerous

Re: Changes to make pylint happy

2008-07-02 Thread Edward K. Ream
On Tue, Jul 1, 2008 at 9:24 PM, TL [EMAIL PROTECTED] wrote: Do you want clean-up of indentation checked into the trunk? My code in mod_tempfilename.py is not indented correctly. That would be good. Thanks. How about a small change? I would like to add a conditional in the open_with

Re: Changes to make pylint happy (continued)

2008-07-02 Thread Edward K. Ream
On Jul 2, 10:26 am, zpcspm [EMAIL PROTECTED] wrote: I've just disabled the warning: W0102 Dangerous default value [] as argument I've found this warning interesting, because I couldn't see the danger. After googling for a while, I found the

Re: Changes to make pylint happy

2008-07-01 Thread Terry Brown
Interesting. I've fixed some errors in cleo, graphed, and UNL (I'm not UNL's author, but I wrote the code it was complaining about). These errors remain: * Module leo.plugins.UNL W0102:129:recursiveUNLSearch: Dangerous default value [0] as argument W0102:129:recursiveUNLSearch: