cbarrete / tree-sitter-ledger

Ledger grammar for tree-sitter

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

tree-sitter-ledger incorrectly parses multiword commodity names

gregorias opened this issue · comments

I've found that the parser doesn't correctly parse multiword commodity names that results in weird highlighting. See "Hoi Zäme" and "Hammer's German Grammar":

incorrect commodity highlighting

An additional issue visible in that screenshot is that the parser doesn't take non-ASCII characters into account.

I'm using the most recent neovim and tree-sitter plugin.

Hey, thanks for reporting that.

Regarding the non-ASCII characters, I'm waiting for tree-sitter/tree-sitter#906 to be done/merged to have proper regexes available for Unicode letters. If this bothers you too much, feel free to submit a PR, I just don't want to personally spend time on a workaround just to throw it away in a few days (hopefully).

I'll fix the multiword commodities though.

This will land in nvim-treesitter when nvim-treesitter/nvim-treesitter#945 is merged :)

Sure thing, thanks!