redhat-developer / redhat-developer/yaml-language-server
Consider extracting yaml-language-service from yaml-language-server
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 1.5k
- Forks
- 352
- Avg merge
- 2d 7h
- Merged PRs (30d)
- 11
Description
Summary
I would like to be able to depend on the language service part without pulling in the language server part and all of its dependencies.
Relevant information
I’m working on monaco-yaml. monaco-yaml depends on the YAML language service part in this package, but not on the language server. Currently a build process is needed to include the language service, but ideally I would like to be able to simply add it as a dependency without pulling in the language server part and its dependencies.
I believe extracting the language service also helps to better separate the two parts. , but I noticed the latest version of the language service also uses some imports from the language server.monaco-yaml still depends on an older version of yaml-language-server
I understand if this gets rejected, because it adds an additional maintenance burden, but I wanted to ask anyway to see what you think of it. I think using a yarn mono repository reduces the additional maintenance a lot.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing how the language service and language server are currently packaged and which imports cross between them. Check the yarn monorepo approach and the dependencies needed by monaco-yaml. Done means the language service can be added independently without pulling in the language server or its dependencies.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100