-
-
Notifications
You must be signed in to change notification settings - Fork 625
docs: add (disabled) diagnostics config to bug report template #2484
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
.github/ISSUE_TEMPLATE/nvt-min.lua
Outdated
@@ -11,6 +11,10 @@ local function load_plugins() | |||
"wbthomason/packer.nvim", | |||
"nvim-tree/nvim-tree.lua", | |||
"nvim-tree/nvim-web-devicons", | |||
|
|||
-- UNCOMMENT for diagnostics issues | |||
-- "neovim/nvim-lspconfig", |
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.
Shouldn't we request native lsp start function call over additional plugin?
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.
Yes please!
Do you know how to get that going?
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.
I guess we can just direct to :h vim.lsp.start
which has an example right 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.
Well... I was overthinking that; I didn't realise it was that simple.
Please test drive the new commented out template.
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 makes total sense. Seeing this I even thought to add a separate bug report template for diagnostics related issues but maybe that's too much. This seems an optimal middle ground.
Drafted during #2460