Skip to content
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

Add support for strongly typed data to be attached to */resolve #250

Closed
wants to merge 2 commits into from

Conversation

david-driscoll
Copy link
Member

This is an initial stab at adding support for strongly typed data to be attached to codelens, completion items, and document links.

Adds support for multiple codelens, completion and document link handlers for each text document.

What this means is that you can have multiple codelens (and completion/document link) handlers for a document selector.

You might have a codelens handler that returns test lenses (and sets up data specific to text lenses). You could then have another handler that returns code lens for references, with a different data element. These will then be automatically merged for you by the server, and resolve requests will be routed to the correct handler.

…pletion items, and document links. Adds support for multiple codelens, completion and document link handers for each text document.
@codecov
Copy link

codecov bot commented Jun 2, 2020

Codecov Report

Merging #250 into master will decrease coverage by 1.23%.
The diff coverage is 50.13%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #250      +/-   ##
==========================================
- Coverage   62.12%   60.89%   -1.24%     
==========================================
  Files         383      388       +5     
  Lines       11483    11720     +237     
  Branches     1361     1327      -34     
==========================================
+ Hits         7134     7137       +3     
- Misses       3782     4105     +323     
+ Partials      567      478      -89     
Impacted Files Coverage Δ
src/Protocol/Document/ITextDocumentSyncHandler.cs 0.00% <0.00%> (ø)
...rc/Protocol/Server/Capabilities/CodeLensOptions.cs 0.00% <0.00%> (ø)
.../Protocol/Server/Capabilities/CompletionOptions.cs 77.77% <0.00%> (-11.12%) ⬇️
...rotocol/Server/Capabilities/DocumentLinkOptions.cs 0.00% <0.00%> (ø)
src/Protocol/Document/ICodeLensHandler.cs 22.27% <22.91%> (-40.58%) ⬇️
src/Protocol/Document/IDocumentLinkHandler.cs 18.18% <25.75%> (-36.74%) ⬇️
src/Protocol/Document/ICompletionHandler.cs 27.47% <26.98%> (-29.43%) ⬇️
src/Protocol/DelegatingHandlers.cs 31.40% <33.17%> (-0.76%) ⬇️
src/Protocol/Models/CompletionList.cs 44.59% <50.94%> (+8.88%) ⬆️
src/Server/Matchers/ResolveCommandMatcher.cs 90.32% <60.00%> (+31.70%) ⬆️
... and 35 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43e946c...9132bb7. Read the comment docs.

@david-driscoll
Copy link
Member Author

#269 superseeds this one, as it has fewer breaking changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant