On 19 juin 06:59, Ned Batchelder wrote:
> I don't think I can make an actual patch, but can I suggest a very
> simple change? This message:
>
> Invalid name "f" for type variable (should match
> [a-z_][a-z0-9_]{2,30}$)
>
> would be simpler and clearer without the word "type". I always read
> it first as "f is not a valid name for a type". Can we shorten it
> to:
>
> Invalid name "f" for variable (should match [a-z_][a-z0-9_]{2,30}$)
Good suggestion, I've just changed so you'll get in case of the above example:
Invalid variable name "f"
So it's even shorter (we decided to drop the regexp pattern as well since it's
not easily readable and may be a long string).
--
Sylvain Thénault, LOGILAB, Paris (01.45.32.03.12) - Toulouse (05.62.17.16.42)
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
_______________________________________________
code-quality mailing list
[email protected]
http://mail.python.org/mailman/listinfo/code-quality
