-
Notifications
You must be signed in to change notification settings - Fork 352
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
add additional tests for coverage, #928
Conversation
… reduce code duplication
Codecov ReportAll modified lines are covered by tests ✅
... and 6 files with indirect coverage changes 📢 Thoughts on this report? Let us know!. |
…hat arose with comment chars with no delimiters for flag like options.
@henryiii I am going to merge this in one day unless you object. (2023-10-7) |
I've already approved so you can merge any time (you don't have to have my approval unless it's potentially controversial or something). And these sorts of cleanups & added tests don't need much review, IMO. |
Add some additional tests to try to get test coverage back to 100%
refactor the positional parsing to reduce code duplication