Email [Python-projects] Re: SPE & PyLint

On Wednesday 22 December à 13:19, Stani Michiels wrote:
> >
> >
> >Hello,
> >
> >I noticed the UML view of SPE v0.7. That's really nice. I saw that you also
> >have support for pychecker and other tools and wanted to suggest 
> >integration
> >of http://www.logilab.org/projects/pylint/ 
> ><javascript:ol('http://www.logilab.org/projects/pylint/');>
> >
> >PyLint is already integrated in emacs, pydev for eclipse and eric (IIRC).
> > 
> >
> Dear Nicolas,
> I'll download it and I'll try it. Of course I'm intrested in integrating 
> it, but actually is more a matter or time. Currently I'm fixing out bugs 
> for the mac and later I want to implement a debugger.  However you could 
> speed up the integration by writing some code for SPE. I guess the 
> implementation will be similar to PyChecker. Therefore you should have a 
> look at _spe/Plugin/PyCheck.py If you need any help, ask! I just tried 
> the pylint-gui.bat, but whatever module I give, it doesn't go further than:
>    C:\Python23\Scripts>rem = """-*-Python-*- script
> 
> Am I missing something?

Herr, I'm definitivly not a windows user so i'm ccing the
python-projects mailing list, maybe someone may help there ?
 
> When I try to run lint.py on _spe, I get the following error:
> ...
> ************* Module _spe.plugins.pychecker2.tests.class
> W:107: Bad indentation. Found 3 spaces, expected 4
[snip message]
> Traceback (most recent call last):
>  File "C:\Python23\Lib\site-packages\logilab\pylint\lint.py", line 956, 
> in ?
>    Run(sys.argv[1:])
[snip traceback]
>  File "C:\Python23\Lib\site-packages\logilab\common\astng\astng.py", 
> line 201,
> in resolve
>    raise ResolveError(name)
> logilab.common.astng.ResolveError: ast

Hum, actually I may have already fixed that problem, but I'll have to
test it to check. I'll try that asap.

> The implementation will be quite similar as pychecker indeed, just 
> parsing the linenumbers. I suppose C,R,W stand for something. If I ship 
> pylint with spe, I need to strip it down to its minimum. Could you make 
> me a list of the files needed. (I saw pylint also depends on the common 
> library.)

The --parseable=y option may be helpfull, with it it produces a more
usual output file:line:message (recognized by editors such as emacs and
vi for instance).

Pylint messages are divised in different categories :
* C for convention violation
* R for refactoring (bad code smell)
* W for warning (bad python habit)
* E for error (much probably bugs)
* F for fatal (internal error preventing some processing)

And Pylint is only depending on the common library, nothing else.

regards
-- 
Sylvain Thénault                               LOGILAB, Paris (France).

http://www.logilab.com   http://www.logilab.fr  http://www.logilab.org

_______________________________________________
Python-Projects mailing list
Python-Projects@lists.logilab.org
http://lists.logilab.org/mailman/listinfo/python-projects