Skip to content

backticks and $(...) aren't highlighted properly #139

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

Closed
dimonomid opened this issue Mar 22, 2014 · 5 comments
Closed

backticks and $(...) aren't highlighted properly #139

dimonomid opened this issue Mar 22, 2014 · 5 comments

Comments

@dimonomid
Copy link

Say, in this command:
vim $(find -name "something")
both vim and find should be highlighted as a command, but only vim is.

@anntzer
Copy link
Contributor

anntzer commented Apr 14, 2015

There's also the process substitution commands, cmd <(cmd), cmd >(cmd) and cmd =(cmd).

@ZevEisenberg
Copy link

Also, commands in double-quoted strings. I was just bitten by this when I ran the following commit message:

git commit -m "Add library attributions Settings page that updates whenever `pod install` is run."

Of course, the pod install was executed, and the output was inserted into my commit message. I wouldn't have run into that problem if zsh-syntax-highlighting had flagged the backtics in the commit message.

@danielshahaf
Copy link
Member

Also, commands in double-quoted strings.

Addressed by c3913e0 for the "$(…)" syntax.

@danielshahaf
Copy link
Member

danielshahaf commented Nov 11, 2016

Summary:

What needs highlighting:

  • Command substitution : echo foo``, : $(echo foo).
  • Command substitution in double quotes: "`echo foo`", "$(echo foo)" — the latter case currently highlights the $ but not the nested command.
  • Process substitution <(foo), =(foo), >(foo).

How to highlight:

@phy1729
Copy link
Member

phy1729 commented Jan 20, 2018

Also, commands in double-quoted strings.

And now addressed for backticks by f16e858. (Although back-quoted-argument is not highlighted by default).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants