-
Notifications
You must be signed in to change notification settings - Fork 337
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
Single file support WIP #5531
Single file support WIP #5531
Conversation
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.
The approach looks good! I think we can go that direction, sorry for so late a review.
@@ -153,6 +154,29 @@ class WorkspaceLspService( | |||
languageClient, | |||
) | |||
|
|||
val fallbackService: FallbackMetalsLspService = { |
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.
Maybe we could create it lazily?
buildTargets: BuildTargets, | ||
)(implicit ec: ExecutionContextExecutorService) | ||
extends Cancelable { | ||
val buildTargetsData = new TargetData |
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.
Is it used?
0267afd
to
d0f78cc
Compare
bdb9373
to
bd2697f
Compare
7600a7e
to
6dfe0ed
Compare
6dfe0ed
to
43c690c
Compare
A fast solution to add single file support.
resolves: scalameta/metals-feature-requests#231
TODO:
scala-cli
serversMetalsLspServer
so it doesn't need a main connection