Fix: Short version of "dry run" option yields an error #88

peritus / bumpversion

Version-bump your software with a single command

https://pypi.python.org/pypi/bumpversion

MIT License

1.5k stars 147 forks source link

Closed inirudebwoy closed 8 years ago

inirudebwoy commented 9 years ago

Hi,

I fixed the issue https://github.com/peritus/bumpversion/issues/88. It also was reproducible with '-m' flag. What do you think?

peritus commented 9 years ago

Thanks for the fix!

I'd probably add a regression test to this before merging.

inirudebwoy commented 9 years ago

Ok, I will update this PR with some tests.

peritus commented 9 years ago

Just one demonstrating the issue would do. Thanks a lot!

inirudebwoy commented 9 years ago

I have added few tests for function split_args_in_optional_and_positional.

peritus commented 8 years ago

I just commited your patch without the whitespace changes as 6ec908f018a827bfae94ed50baf241461eba7bcb to master.

Your contribution is highly appreciated, thanks!