-
Notifications
You must be signed in to change notification settings - Fork 119
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
Trailing and leading quote characters are removed #201
Labels
Comments
mefcorvi
changed the title
Question about quoting again
Trailing and leading quote characters are removed
Sep 1, 2019
@mefcorvi in an ideal world |
We should distinguish between parsing a
So the change would have to be done in "tokenize-arg-string.js", and not in our parser. |
1 task
Thanks for the report, and the simple reproduction. I can no longer reproduce the reported issue. I suspect it was fixed by #407. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version 13.1.1
Description
Trailing and leading quote characters are removed from the values. If I want to pass a
'string'
as a value I need to triple-quote it as"''string''"
or\'''string\'''
when I'm specifying that in the shell.Expected
Actual
This probably relates to #138, #180. Definitely relates to #145 and #153.
I would suggest introducing a new
keepQuotes
/trimQuotes
option to support the all edge cases.The text was updated successfully, but these errors were encountered: