-
Notifications
You must be signed in to change notification settings - Fork 54
Add Kakoune and Helix configuration #181
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
Conversation
Please add hard line breaks after each sentence, it makes reading diffs much easier. And I'm not sure about the helix remark/emoji. I don't mind them that much, but they don't really fit the style used in the rest of the docs. |
632f4ab
to
05ce286
Compare
Indeed, done.
I might have been a bit too comfortable here 😅 Let's say it was late yesterday evening! I removed it. |
manual.adoc
Outdated
@@ -427,6 +427,40 @@ Then click on apply, and restart the LSP server for your rust project. | |||
|
|||
https://gitlab.com/cppit/jucipp[juCi++] has built-in support for the language server protocol, and since version 1.7.0 offers installation of both Rust and rust-analyzer when opening a Rust file. | |||
|
|||
=== Kakoune | |||
|
|||
https://kakoune.org/[Kakoune] editor support LSP with the help of https://github.com/kak-lsp/kak-lsp[`kak-lsp`]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://kakoune.org/[Kakoune] editor support LSP with the help of https://github.com/kak-lsp/kak-lsp[`kak-lsp`]. | |
https://kakoune.org/[Kakoune] supports LSP with the help of https://github.com/kak-lsp/kak-lsp[`kak-lsp`]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in ebca994.
manual.adoc
Outdated
|
||
=== Helix | ||
|
||
https://docs.helix-editor.com/[Helix] editor does support LSP by default. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://docs.helix-editor.com/[Helix] editor does support LSP by default. | |
https://docs.helix-editor.com/[Helix] supports LSP by default. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in ebca994.
bors d+ changelog fix (first contribution) add install instructions for Kakoune and Helix. |
✌️ woshilapin can now approve this pull request. To approve and merge a pull request, simply reply with |
bors r+ |
Configuration problem: |
First time I interact with bors. I thought I was allowed to do |
Sorry I didn't notice it, can you port your changes to https://github.com/rust-analyzer/rust-analyzer/blob/master/docs/user/manual.adoc? There is a comment at the top:
|
New PR opened in rust-lang/rust-analyzer#11535. |
11535: Add Kakoune and Helix configuration r=lnicola a=woshilapin Follow-up of rust-analyzer/rust-analyzer.github.io#181 Fixes rust-analyzer/rust-analyzer.github.io#180 cc `@lnicola` Co-authored-by: Jean SIMARD <woshilapin@tuziwo.info>
Fixes #180.
I choose to put links to existing official documentation instead of copy/paste what to do with the risk of being out-of-date (especially for these young editors where things move often).