-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat: go to implementation/references/declaration/type definition with menu #291
feat: go to implementation/references/declaration/type definition with menu #291
Conversation
b4ca79a
to
12cea4b
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #291 +/- ##
==========================================
+ Coverage 21.34% 21.83% +0.49%
==========================================
Files 245 259 +14
Lines 8372 8655 +283
Branches 1548 1570 +22
==========================================
+ Hits 1787 1890 +103
- Misses 6249 6408 +159
- Partials 336 357 +21 ☔ View full report in Codecov by Sentry. |
0b62a41
to
16fb8a5
Compare
16fb8a5
to
630f051
Compare
5fcd727
to
e9a8984
Compare
@@ -42,14 +42,13 @@ Current state of [Language Features]( https://microsoft.github.io/language-serve | |||
* ✅ [completionItem/resolve](https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#completionItem_resolve) (see [implementation details](#completion-item-resolve)) | |||
* ✅ [textDocument/signatureHelp](https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_signatureHelp) (see [implementation details](#signature-help)) | |||
* ✅ [textDocument/publishDiagnostics](https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_publishDiagnostics) (see [implementation details](#publish-diagnostics)) | |||
* ✅ [textDocument/definition](https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_definition) |
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.
why did you remove it? definition != type definition
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.
Indeed but I removed it to avoid having duplication
e9a8984
to
091499d
Compare
menu Signed-off-by: azerr <[email protected]>
feat: go to implementation/references/declaration/type definition with menu