-
Notifications
You must be signed in to change notification settings - Fork 288
Labels
good first issueGood for newcomersGood for newcomerslanguage-serverIssues specific to our IDE integration rather than type checkingIssues specific to our IDE integration rather than type checking
Description
Describe the Bug
For the deprecated error code, we should emit LSP diagnostics w/ the deprecated tag: https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#diagnosticTag
That way, the editor can render a strikethrough over the function name. We should be careful what position we emit the diagnostic in. I think the strikethrough should only cover the function name, not the arguments.
Sandbox Link
No response
(Only applicable for extension issues) IDE Information
No response
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomerslanguage-serverIssues specific to our IDE integration rather than type checkingIssues specific to our IDE integration rather than type checking