The Wayback Machine - https://web.archive.org/web/20201106030743/https://github.com/microsoft/language-server-protocol/issues/926
Skip to content

/ language-server-protocol

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

Jump to bottom

Add support for links in diagnostics #926

Closed
dbaeumer opened this issue Feb 18, 2020 · 5 comments
Closed

Add support for links in diagnostics #926

dbaeumer opened this issue Feb 18, 2020 · 5 comments
Labels
feature-request
Milestone
3.16

Comments

@dbaeumer
Copy link
Member

@dbaeumer dbaeumer commented Feb 18, 2020

VS Code added API to support links in diagnostics. We should investigate if we adopt this in LSP.
@dbaeumer dbaeumer added this to the 3.16 milestone Feb 18, 2020
@egamma egamma mentioned this issue Feb 18, 2020
Closed
42 of 54 tasks complete
@Razzeee
Copy link
Contributor

@Razzeee Razzeee commented Feb 18, 2020

I'm currently showing urls via diagnostics, they are just not linked.
@KamasamaK
Copy link
Contributor

@KamasamaK KamasamaK commented Feb 18, 2020

Relevant VS Code issue is microsoft/vscode#11847
@owenv owenv mentioned this issue Feb 25, 2020
Merged
@larshp larshp mentioned this issue Mar 11, 2020
Closed
@Krzysztof-Cieslak Krzysztof-Cieslak mentioned this issue Apr 10, 2020
Open
@larshp larshp mentioned this issue Apr 12, 2020
Open
@dvandra
Copy link

@dvandra dvandra commented May 26, 2020

Is there any further updates on this feature-request?
@Razzeee
Copy link
Contributor

@Razzeee Razzeee commented May 26, 2020

I think the current state is it being proposed via microsoft/vscode-languageserver-node@fa9ccc4

https://github.com/microsoft/vscode-languageserver-node#3160-next1-protocol-510-next1-json-rpc-620-next1-client-and-620-next1-server
@dbaeumer
Copy link
Member Author

@dbaeumer dbaeumer commented Nov 5, 2020

This got implemented and is documented in the 3.16 spec.
@dbaeumer dbaeumer closed this Nov 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-request
Projects
None yet
Milestone
3.16
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@dbaeumer @Razzeee @KamasamaK @dvandra
You can’t perform that action at this time.