-
Notifications
You must be signed in to change notification settings - Fork 2k
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
docs: delete lspconfig-keybindings section #3175
docs: delete lspconfig-keybindings section #3175
Conversation
This section is way too verbose and encourages sloppy copy-pasting. Nvim already ships help documentation for configuring the LSP client, so instead of duplicating that information, point users toward upstream docs instead.
|
||
- {settings} `table <string, string|table|bool>` | ||
Prefer using an |LspAttach| autocommand handler instead. |
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.
nitpick: README mentions on_attach
instead of LspAttach, might be worth changing it there.
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.
Done d3d4b25
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.
praise: well done
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.
💯
This is a bad decision - the keybinds section provided a great jumping-off point for 99% of users, and over time has become the de facto resource for bootstrapping a new configuration - when you google something like "neovim lsp go to definition" virtually all resources lead back to this README, which has now been stripped out of any useful content. |
Pretty much all keybindings have been upstreamed to neovim and is the default now. |
This section is way too verbose and encourages sloppy copy-pasting (not to mention it is out of date). Nvim already ships help documentation for configuring the LSP client, so instead of duplicating that information, point users toward upstream docs instead.
Fixes: #3174
Related: #3122