Feature Positional parameters #96
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Empty option prefixes are not allowed * Short options need to have a length of 1 * Return results by yielding * Add API obsolete safety test * Add second implementation of argument manager that will work better * Add command context to keep track of the current command * Add logging * Test run with ArgumentManager2 implementation * Add logging in tests * Implement input parsing feature v2 fixes #89 * Use new ArgumentManager everywhere * Remove old ArgumentManager * Inject IArgumentManager * Remove non async versions
Matthiee
added
enhancement
Codecov Report
@@ Coverage Diff @@
## master #96 +/- ##
==========================================
+ Coverage 86.48% 86.54% +0.06%
==========================================
Files 43 45 +2
Lines 1236 1249 +13
Branches 211 212 +1
==========================================
+ Hits 1069 1081 +12
+ Misses 106 104 -2
- Partials 61 64 +3 |
Matthiee
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #54