Hi Martin,

On 31 août 17:50, Martin Pool wrote:
> Great!  Here's a first cut of a patch for it.  I have not yet added
> symbols for all warnings but I think it could stand to have some
> review.
> Example output:
> 
> % python lint.py -rn -sy lint.py
> No config file found, using default configuration
> ************* Module pylint.lint
> C(line-too-long):903,0: Line too long (91/80)
> W(fixme):555,0: XXX hack, to be correct we need to keep module_msgs_state
> C(no-space-after-comma): 50,0: Comma not followed by a space
> C(missing-docstring): 71,0:_get_python_path: Missing docstring

I would have expected the message string to rather replace the message,
similarly as e.g. lintian:

C:903,0:line-too-long: 91/80
W:555,0:fixme: XXX hack, to be correct we need to keep module_msgs_state
C: 50,0:no-space-after-comma: 
C: 71,0:missing-docstring:

or

C-line-too-long:903,0: 91/80
W-fixme:555,0: XXX hack, to be correct we need to keep module_msgs_state
C-no-space-after-comma: 50,0:
C-missing-docstring: 71,0:

or 

903,0:C-line-too-long: 91/80
555,0:W-fixme: XXX hack, to be correct we need to keep module_msgs_state
 50,0:C-no-space-after-comma: 
 71,0:C-missing-docstring:


My preference going to the later.

implementation tip: this should probably be a new reporter

-- 
Sylvain Thénault, LOGILAB, Paris (01.45.32.03.12) - Toulouse (09.54.03.55.76)
Formations Python, Debian, Méth. Agiles: http://www.logilab.fr/formations
Développement logiciel sur mesure:       http://www.logilab.fr/services
CubicWeb, the semantic web framework:    http://www.cubicweb.org
_______________________________________________
Python-Projects mailing list
Python-Projects@lists.logilab.org
http://lists.logilab.org/mailman/listinfo/python-projects

Reply via email to