`addArgument([...],{defaultValue:0})` does not work about argparse HOT 4 CLOSED

@hpaulj thanks a lot. Could you do it as pull requests and add tests? That will greately speedup accepting fixes.

from argparse.

On Sun, Dec 9, 2012 at 8:49 AM, Vitaly Puzrin [email protected]:

@hpaulj https://github.com/hpaulj thanks a lot. Could you do it as pull
requests and add tests? That will greately speedup accepting code,

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/22#issuecomment-11172787.

I've committed a number of changes to my fork. I need to read up on pull
requests.

Paul

from argparse.

Ah, i see now. It worth to follow simple rules:

1. Do each fix in separate branch (to make separate pull requests). If one commit will be rejected/postponed, that will not delay others.
2. Add test to each fix. We are not fans of TDD, but all fixes should be covered.

from argparse.

Fixed in ither commit

from argparse.