This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eric.araujo
Recipients amcnabb, docs@python, eric.araujo, georg.brandl, pycurry, r.david.murray, sandro.tosi
Date 2010-10-01.21:14:34
SpamBayes Score 8.279817e-06
Marked as misclassified No
Message-id <1285967677.34.0.756996376336.issue5088@psf.upfronthosting.co.za>
In-reply-to
Content
Thanks for the patch.  Some remarks:

>  :attr:`~Option.dest` variable, that already contains the default value
I would have used “which” here, but I’m not a native speaker.

> not replaced (contrary to what one can think)
I’d have used a comma, not parens.

> This behaviour is clear
“makes sense” sounds better to me.

> think that with a default value :attr:`~Option.dest` is a list
I suggest a comma after “value”.

> any additional option is appended to that list
s/option/value/

Let me use this reply to welcome you in the bug tracker.  I hope you get warn fuzzy feelings when your patches are accepted or your comments acted upon.  I’m also looking forward for a better Python-Debian relationship.  (Cultural note: It’s not usual to say hello and regards in messages on this tracker.  I did it too at first but was told it was unnecessary.)  :)
History
Date User Action Args
2010-10-01 21:14:37eric.araujosetrecipients: + eric.araujo, georg.brandl, amcnabb, r.david.murray, pycurry, sandro.tosi, docs@python
2010-10-01 21:14:37eric.araujosetmessageid: <1285967677.34.0.756996376336.issue5088@psf.upfronthosting.co.za>
2010-10-01 21:14:35eric.araujolinkissue5088 messages
2010-10-01 21:14:34eric.araujocreate