textmate / shellscript.tmbundle

TextMate support for ShellScript
43 stars 29 forks source link

Improve shell syntax highlighting #45

Open aeschli opened 7 years ago

aeschli commented 7 years ago

From @bfred-it on October 6, 2017 11:24

This is the current highlighting for a .sh file, which is essentially zero. Can this be extended to at the very least highlight the command name and --flags? That'd go a long way

no-syntax highlighting

Example from fishell (no flag highlight but it underlines existing file paths):

fishell example 1
Extension Author (truncated) Version
EditorConfig Edi 0.10.1
beautify Hoo 1.1.1
fish Ted 0.0.4
vs-code-bitkompagniet-require bit 0.7.0
vscode-svgviewer css 1.4.3
vscode-eslint dba 1.3.2
gitlens eam 5.4.1
vscode-npm-script eg2 0.2.1
code-runner for 0.7.2
Untabify ili 0.0.2
search-node-modules jas 1.2.0
vscode-gutter-preview kis 0.11.1
vscode-lebab mrm 2.5.0
sublime-keybindings ms- 2.9.1
typescript-javascript-grammar ms- 0.0.24
debugger-for-ios-web msj 0.1.2
node-modules-resolve nau 1.0.2
peep nwa 0.0.5
vscode-icons rob 7.14.0
linter-xo sam 2.1.3
trailing-spaces sha 0.2.11
stylelint shi 0.29.1
vscode-fileutils sle 2.7.0
ayu tea 0.8.0
shellcheck tim 0.1.0
ghosttext tok 1.0.2

Copied from original issue: Microsoft/vscode#35707

fregante commented 7 years ago

Actually a duplicate of #7, which is unfortunately open since 2011 :(

tbates commented 7 years ago

shall we merge these two issues? (#7 and #45?)

My bundle fork is here It has some added commands. It would be great to import an open source python or other library which does this. Would need support from @infininight and @sorbits, as well as the current (is there one? it's been so long...) bundle maintainer. I gave up last time. Very happy to work on this again though!

Currently getting an error trying to pull the most recent updates from the textmate managed bundle to merge with mine. Will explore

git -c diff.mnemonicprefix=false -c core.quotepath=false -c credential.helper=sourcetree fetch upstream 
error: cannot open .git/FETCH_HEAD: Permission denied
icetbr commented 2 years ago

This should help a bit better-shellscript-syntax