-
Notifications
You must be signed in to change notification settings - Fork 1
tame the gigantic conflicts diagnostic #7
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
Comments
FWIW, I filed microsoft/language-server-protocol#1458 and microsoft/language-server-protocol#1461 related to this, I hestitate to do it for a few reasons:
Anyhow I just don't realistically see all the conflict information fitting within the UI that diagnostics are currently limited to. The writing a report directly to the filesystem and linking to that, does have the benefit of being something we can emit now Option 2: Adding an lsp extension, I think we'd have to basically emit an entire replacement for |
The gigantic conflicts diagnostic needs a lot of experimentation.
In theory each state may be separated out into a 'related diagnostic',
providing a jump location for each individual state.
But diagnostics are typically fairly short and editor UI's reflect that.
vscode even with the, diagnostic output window maximized doesn't read easily,
nvim
lsp.diagnostics.open_float()
doesn't scroll.I'm not certain diagnostics are actually going to be the right thing for this information,
It may be that we want an LSP extension which provides a list of jump targets.
The text was updated successfully, but these errors were encountered: