-
Notifications
You must be signed in to change notification settings - Fork 676
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
New completion service #3986
New completion service #3986
Conversation
… the roslyn backend for a better overall completion experience.
Codecov Report
@@ Coverage Diff @@
## master #3986 +/- ##
==========================================
+ Coverage 85.96% 85.99% +0.03%
==========================================
Files 60 60
Lines 1852 1856 +4
Branches 214 214
==========================================
+ Hits 1592 1596 +4
Misses 200 200
Partials 60 60
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
I added a new test for override completion that should hopefully fail until the roslyn dependency is updated. If it doesn't fail, I'll be concerned... |
Finally, Waiting for merge and new release |
@JoeRobich this now has an option for import completion as well. |
Co-authored-by: Joey Robichaud <[email protected]>
thanks for adding the VS Code option for |
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.
Thanks Fred!
Use the new completion service being added in OmniSharp/omnisharp-roslyn#1877. This enables things like override completion, as well as overall faster completion due to less filtering and preprocessing being done on the server side.
Fixes #1044.
Fixes #3687.