-
Notifications
You must be signed in to change notification settings - Fork 362
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error work #56
Error work #56
Conversation
Codecov Report
@@ Coverage Diff @@
## master #56 +/- ##
==========================================
+ Coverage 99.84% 99.92% +0.07%
==========================================
Files 9 9
Lines 1311 1314 +3
Branches 259 253 -6
==========================================
+ Hits 1309 1313 +4
+ Misses 2 1 -1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #56 +/- ##
========================================
+ Coverage 99.84% 100% +0.15%
========================================
Files 9 9
Lines 1311 1280 -31
Branches 259 246 -13
========================================
- Hits 1309 1280 -29
+ Misses 2 0 -2
Continue to review full report at Codecov.
|
This cleans up Error throwing in several cases.
ArgumentMismatch
if that number was not received (changes old error in some cases, finally correctly throws an error in the 0 received for 1 case)This also unifies the old
_parse_short
and_parse_long
into a single function,_parse_arg
, reducing a lot of redundancy.