Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

[LanguageServer] Support the case that rootPath is null #57

Closed
spoenemann opened this issue Jul 27, 2016 · 4 comments · Fixed by #372
Closed

[LanguageServer] Support the case that rootPath is null #57

spoenemann opened this issue Jul 27, 2016 · 4 comments · Fixed by #372
Labels
enhancement

Comments

@spoenemann
Copy link
Member

When VS Code opens a file without opening a directory first, the language server is initialized with rootPath = null. This case should also be supported by the Xtext server (we would have a single resource then).
@cdietrich
Copy link
Member

cdietrich commented Jul 27, 2016 •
edited
Loading

i saw this before. so we want to support this as well?

@svenefftinge
Copy link
Member

When rootPath is null, we don't index and start with an empty index and resourceset.
Simply waiting for didOpen notification

@spoenemann spoenemann added this to the release_v2.11 milestone Oct 6, 2016
@colameo colameo mentioned this issue Oct 6, 2016
Closed
@spoenemann
Copy link
Member Author

@svenefftinge is this already done?

@cdietrich
Copy link
Member

imho this is still open

@spoenemann spoenemann removed this from the release_v2.11 milestone Jan 20, 2017
@cdietrich cdietrich mentioned this issue Feb 14, 2017
Open
@kittaakos kittaakos mentioned this issue Feb 22, 2017
Closed
spoenemann added a commit that referenced this issue Jun 30, 2017
@spoenemann
Fixed #57: Create an in-memory workspace configuration when no rootUR…
d424d9f 
…I is given
@spoenemann spoenemann mentioned this issue Jun 30, 2017
Merged
spoenemann added a commit that referenced this issue Jul 5, 2017
@spoenemann
#57: Added a test with file scheme
55e3eef
spoenemann added a commit to kieler/elk-live that referenced this issue Jul 6, 2017
@spoenemann
Removed workaround for eclipse/xtext-core#57 (see eclipse/xtext-core#372
43d3de1 
)
@dhuebner dhuebner mentioned this issue Nov 21, 2018
Merged
@chadkennedyonline chadkennedyonline mentioned this issue Dec 25, 2021
Open
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Create an in-memory workspace configuration when no root URI is given eclipse/xtext-core
3 participants
@cdietrich @svenefftinge @spoenemann