-
Notifications
You must be signed in to change notification settings - Fork 27
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
Improve code splitting #416
Conversation
…typescript language service
…ew files it allows importing the service identifier WITHOUT importing all the code in the same module
@kaisalmen 1610bf3 may impact your monaco-editor-wrapper, you may want to import the new monarch service override when textmate is not used Related to microsoft/monaco-editor#4463 |
38241b2
to
6a874ae
Compare
@CGNonofr Thank you, for seeing that. And thank you for providing the additional service override. I am building this branch locally right now and will test it today. This is a big change, but welcome one. Do you want to create another pre-release or do you want to release a new version soon (as soon as I approve)? What do you prefer? As usual I will test this with the wrapper and the all examples in the mlc repo and provide feedback. From my point of view we can directly release new version. I also wanted to implement another improvement in the wrapper, but did not had the time, yet. |
I like to move fast and break things! It will be a new minor version, anyone encountering issues can stay on the previous release |
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.
LGTM. Will test this with the usual suspects once it released.
🎉 This PR is included in version 4.5.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
@CGNonofr I needed to some extra packages otherwise vite dev server complains: https://github.com/TypeFox/monaco-languageclient/pull/648/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519 Apart form that it looks good. I opened TypeFox/monaco-languageclient#648 |
well spotted... |
Extract the service identifiers into their own files
Many services (especially those not browser or native specific) have their implementation in the same file as the service identifier, making it impossible to get the service identifier without the implementation.
In monaco-vscode-api, we import all service identifiers from
missing-services.ts
to create an empty overridable implementation of them, currently leading to importing the VSCode implementation as well, making the bundle bigger is the treeshaker is not able to remove them, and making the library tarball bigger anywayAlso update to monaco-editor 0.48 which was released in the meantime but there is no change expected