-
Notifications
You must be signed in to change notification settings - Fork 111
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
LSP: make use of relatedInformation
for richer diagnostics
#258
Conversation
relatedInformation
for richer diagnostics withrelatedInformation
for richer diagnostics
relatedInformation
for richer diagnosticsrelatedInformation
for richer diagnostics
very cool |
this seems to solve this? #254 that would be so awesome |
@thymikee is there a way to try that out? |
@macrozone yup, you're right! |
Just merged facebookarchive/flow-language-server#82 and published it as |
relatedInformation
for richer diagnosticsrelatedInformation
for richer diagnostics
Thanks @wbinnssmith, updated! cc @orta |
Yep, cool! |
Shipped in 0.8.2 |
very nice, this works very well! Btw. should we consider useLSP to be the default? at the moment, it has to be turned on manually |
Not sure if folks gonna like it, it lacks couple of existing options like run on edit instead of save. But I got couple of folks interested in contributing, so hopefully we'll get to feature parity quicker. |
Just updating the deps to get it working.
This PR is currently blocked by facebookarchive/flow-language-server#82. After this is released, we can update the
flow-language-server
dep once again and get this feature in.Fixes #254
Here's a preview of richer diagnostics (note the links):