-
Notifications
You must be signed in to change notification settings - Fork 52
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
Add syntax highlighting and LSP configuration for Vim/Neovim #1518
Conversation
@gaganchandan thanks so much! I can't really verify whether this works but it looks good to me overall. There are two other related things we'll want to update:
|
Should I remove the changes made to the README from the PR? About the tests, I took a look and it seems doable for me. Can I ask here if I run into any issues, or would you prefer a different medium, say, the IRC? |
No, you only changed
Great! Either one is fine. Asking here makes sense since the questions and answers will be archived and more easily discoverable for, say, someone adding support for additional editors in the future. But IRC is fine too, e.g. if you have quick questions that seem like they don't belong here. |
The tests are working locally on my branch, but they are failing here because |
Hmm, that shouldn't make a difference. I suspect the tests are actually failing because |
You were right @byorgey. I made the changes and all the tests passed. |
@gaganchandan Should we add |
Yes, it does make sense to include it. I'll just add it real quick. Are there any more changes you would like? |
No, this looks great, thanks! The CI failure looks spurious, I'm re-running that job now and hopefully this should merge once the CI passes. |
@gaganchandan , congrats on having your first PR accepted to Swarm! We appreciate the contribution and we're really glad to welcome you as part of the community. In a minute we will send you an invite to the repo (see https://github.com/swarm-game/swarm/blob/main/CONTRIBUTING.md#i-have-push-access-to-the-swarm-repository-now-what ). If you use IRC, feel free to also join the |
Thank you so much! |
@gaganchandan thanks a lot, I just set it up it and it works great! 🥳 |
This adds two files. The first is the syntax file responsible for basic highlighting. The highlight categories are based roughly on those seen in the Emacs file (swarm-mode.el). The second is the file for configuring the language server. Since it is based on Neovim's native LSP client, it only works with Neovim and not Vim.
README.md
in theeditor
folder has also been updated to include instructions for setting these up.