search

akamensky / argparse

Argparse for golang. Just because `flag` sucks
MIT License
604 stars 62 forks source link

panic on addArg failure, and tests to check it #57

Closed goofinator closed 4 years ago

goofinator commented 4 years ago

Is something like this normal to test addArg success?

By the way: after adding this check, 2 errors occurred in the test functions. 1 - no long name, while Case sensitivity was tested. 2 - short name "i1".

akamensky commented 4 years ago

LGTM