Berker Peksag added the comment:

The patch needs tests and documentation.

> +    parser.add_argument('--no-ensure-ascii', action='store_true', 
> default=False,

I'd go with ``action='store_false', default=True``.

----------
nosy: +berker.peksag
stage:  -> patch review

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue27413>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to