Feat: Improve existing syntax highlighting

[WilsonZiweiWang]

Improve existing syntax highlighting

[gwerderm]

I just stumbled across this pull request and found some things i would do slightly different. Also (I didn't comment this in the code) i was asking myself, what might has been the reason for you to remove the includes of "source.python" and "source.shell". I think using them could make it easier to write the code, while being sure that the code inside functions gets highlighted properly. But maybe I am wrong with that?

[WilsonZiweiWang]

I just stumbled across this pull request and found some things i would do slightly different. Also (I didn't comment this in the code) i was asking myself, what might has been the reason for you to remove the includes of "source.python" and "source.shell". I think using them could make it easier to write the code, while being sure that the code inside functions gets highlighted properly. But maybe I am wrong with that?

Thanks for the feedback, I pushed some new changes before I saw your comments. The new one should have fixed most of the issues you mentioned.

As for source.python and source.shell, they caused me some trouble at the beginning because some of the rules have conflicts. So I removed them. But you are right that they will save me some effort. To take advantage of them, I figure it would be better to include them in scopes that only Python-like or Shell-like code is present instead of in the global.

[idillon-sfl]

Looks quite good. Just a bunch of corner cases I found:

# python should be colored as a variable
python = "test"

# inherit should be colored as a function name
python inherit () {}

# 'then' should be same color as 'if'
if [ somecondition ] ; then

# no idea what should happen, but this is weird
TEST = python inherit () {}

# This one too is weird
inherit from export inherit from test inherit test from inherit

That being said, don't hesitate to just merge and look at it later.

I would also suggest to use a json prettifier to set the indentation to 2 spaces so we can track the changes better.