Support dockerfiles

[midnightexigent]

I couldn't get it to look as neat as the original

Here is how this PR looks

I am not good enough with tree-sitter.. Anyone has any ideas ?

[archseer]

To get shell highlighting to work, you need to add an injections.scm:

((comment) @injection.content
 (#set! injection.language "comment"))

((shell_command) @injection.content
 (#set! injection.language "bash"))

[midnightexigent]

Thanks for the help

That didn't quite do the trick, I needed an injections.scm that looks like this

((comment) @injection.content
 (#set! injection.language "comment"))

((shell_command) @injection.content
 (#set! injection.language "bash"))

((shell_fragment) @injection.content
 (#set! injection.language "bash"))

Which gives to get the following

Also

((shell_fragment) @injection.content
 (#set! injection.language "bash"))

on its own works also

Does that look okay to you ? If yes I can submit that

[pickfire]

If I am not mistaken, you can reuse the highlights context.

helix/runtime/queries/rust/injections.scm

Lines 1 to 2 in f16651b

	([(line_comment) (block_comment)] @injection.content
	(#set! injection.language "comment"))

[archseer]

@midnightexigent Yeah we want all three clauses, not just the last one (since it's targetting different nodes).

And you can merge the shell_ ones as pickfire proposed:

((comment) @injection.content
 (#set! injection.language "comment"))

([(shell_command) (shell_fragment)] @injection.content
 (#set! injection.language "bash"))

To improve the highlighting further we can compare the bash/highlights.scm with the changes in nvim-treesitter, right now we just copy-pasted the one provided by tree-sitter-bash but it can be further improved.

[midnightexigent]

Done

[archseer]

Thanks! 🎉