-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make a Helix PR to add the Vim language server and grammar #86
Comments
I was able to get the language server working by tweaking the filetypes, but no luck on grammars yet. Here's another config to reference: |
Closes #86 We have the language server working, but no luck yet on the grammar. May require PRing into Helix to add it, but it'd be difficult to get working if I can't have it working on my machine.
Ignore all the spam above. |
I put the language server stuff on main, since it was stable enough and got what I needed working. I had no luck with following what other people did to get the grammar working. Needs further investigation. |
I've been setting up custom Vim binds for Obsidian recently. It's not really suitable to here, since Obsidian isn't one for declarativeness. I'm using an Obsidian plugin that lets me add a custom vimrc to Obsidian, and I've been setting up lots of Helix binds to match my preferences.
However, Helix doesn't understand how to highlight the file, because it doesn't include the tree-sitter for vim files.
If all was right in the world, I could just pass in the custom grammar to the
programs.helix
module, linking directly to the existing tree-sitter grammar. That's done perfectly here.But woe, all is not so simpleInstead, I think PRing the Vim grammar into https://github.com/helix-editor/helix/blob/master/languages.toml is best. It seems relatively simple to include, and will allow other people to be free of issues when it comes to this.
While I definitely want to make that PR, I also want to try out the solution seen here, where they worked around Helix being unable to support custom grammars by just placing it in with
xdg.configFile
directly. Seems fun and cool, and if it works, I could share this as a solution on the home-manager issue.The text was updated successfully, but these errors were encountered: