-
Notifications
You must be signed in to change notification settings - Fork 37
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
Fix main dependencies #520
Conversation
extract code to import only necessary stuff instead of whole modules from core code
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
🎉 This PR is included in version 10.1.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
The xterm.js appears again in the output folder with the 11.1.0 version |
Are you sure? => https://www.npmjs.com/package/@codingame/monaco-vscode-api?activeTab=code there is no xterm |
then please run |
Ok I think I understand, it's not pulled by the main but by the extensions service override, which is a default one |
fixed in #537 |
Don't want to be the bearer of bad news, but now i get an error when i try to build my solution:
Do i have to explicitly externalize the package as vite suggests? Thank you very much for your efforts. |
Hum that's no good news! No, you should probably install them in the meantime |
fix #518