-
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
Add single string parsing #186
Conversation
…string. add the ability to deal with a single string in the parse command and handle quoted string appropriately
I didn't see any place in the readme where it would be appropriate to add a discussion of the function. So I didn't do anything regarding that or in the What's new file. I can add something if you would like. |
Codecov Report
@@ Coverage Diff @@
## master #186 +/- ##
==========================================
- Coverage 100% 99.94% -0.06%
==========================================
Files 12 12
Lines 1818 1841 +23
==========================================
+ Hits 1818 1840 +22
- Misses 0 1 +1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #186 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 12 12
Lines 1818 1849 +31
=====================================
+ Hits 1818 1849 +31
Continue to review full report at Codecov.
|
…ing and handling of extra spaces
I have a few minor consistency changes in mind, but I'll go ahead and merge, then make them. I'll add at least a note in the change log. |
One suggestion: Most of the contents of the |
If merged this pull request will add a function to parse a single string containing all the arguments and optionally the program name. The function splits the single string into a vector and then uses the existing parse function. Basically a wrapper around the split function to deal with the program name and some edge cases that can arise when everything is in the same string.
The PR also includes test code for the functionality and
a modification of the split_up function to handle backquote '`' characters and quoted strings