Implemented highlighting for embedded variables #586

[weltings]

Embedded variables were not highlighted in BDD type statements like:

Financial transaction ${type} should be completed with amount ${amount}

This is now implemented, see also #586

[fabioz]

It doesn't seem like this pull request is correct...

It's including the code for what was already merged. It's the same branch you pushed previously (and it has the same code that's already in master).

Note that the proper approach is checking out your master branch (which should always try to keep in sync with the master branch from https://github.com/robocorp/robotframework-lsp because usually your code will be squashed to a single commit when merging, so, you can't keep using the same branch) then update it (this can be done with git pull https://github.com/robocorp/robotframework-lsp.git master), then create a new branch, do your changes there and provide a pull request for that new branch.

As there's no new code in this branch I'm going to close it (waiting for the proper pull request...).

[fabioz]

p.s.: as you probably do have those commits locally, do the steps above to get a clean branch based on master and then use git cherry-pick to get the related commits in your new branch.

[weltings]

Thanks for the tips