-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Allows config:set to set parameters values with spaces #702
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
Conversation
|
Hum, it does not work over SSH, I'll figure out a way to fix this. |
|
Looking forward to this :) |
|
Merged in c3df79f. Thanks for the pull request! |
@josegonzalez Did you fix this by some other method? @jazzz Did you figure out how to fix support over ssh? |
|
It seemed like his third commit fixed it - and seems to work for other special characters too. I was able to set json as an env variable, for example. |
|
Great! :) |
|
Just did a This works: This does not: (note the space) Am I missing something? |
|
Something must have broken recently re: config setting. We're working on getting tests up and running again, but for now #820 is tracking the issue. |
|
Thanks @josegonzalez |
|
+1 on this |
This fixes #482, and there is a test.