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

bash commands need to be on their own line #41

Closed
odino opened this Issue Dec 19, 2018 · 2 comments

Comments

Projects
None yet
1 participant
@odino
Copy link
Collaborator

odino commented Dec 19, 2018

Parser could use some ❤️

@odino odino added the bug label Dec 19, 2018

@odino odino modified the milestones: future, cauldron, preview-3 Dec 19, 2018

@odino odino modified the milestones: preview-3, cauldron Dec 27, 2018

@odino

This comment has been minimized.

Copy link
Collaborator Author

odino commented Jan 27, 2019

One idea is to not allow } on the same line of a command.

@odino

This comment has been minimized.

Copy link
Collaborator Author

odino commented Jan 27, 2019

Another one is to instead use backticks. They would fundamentally solve every known escaping problem we have, so they might be a good compromise esp. since shells use backticks as well for very similar purposes.

/Cc @ntwrick

@odino odino modified the milestones: cauldron, 1.1.x Jan 27, 2019

@odino odino self-assigned this Jan 30, 2019

odino added a commit that referenced this issue Jan 31, 2019

Added support for backticked commands `` `command` ``, kinda closes #41
For now we can probably live with the original syntax (`$()`)
having to be on its own line.

odino added a commit that referenced this issue Jan 31, 2019

Added support for backticked commands `` `command` ``, kinda closes #41
For now we can probably live with the original syntax (`$()`)
having to be on its own line.

@odino odino closed this Jan 31, 2019

odino added a commit that referenced this issue Feb 2, 2019

Added support for backticked commands `` `command` ``, kinda closes #41
For now we can probably live with the original syntax (`$()`)
having to be on its own line.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment