-
-
Notifications
You must be signed in to change notification settings - Fork 182
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
Updated to [email protected] #575
Conversation
Are you sure there is no duplicated package? |
Observation: Updating the overrides usually requires to nuke both
At least not |
I would be surprised there is that error here and not in monaco-vscode-api demo. |
It happens when I use |
It doesn't ring a bell to me |
I haven't found a smoking gun either last night, but I also didn't search deeper. I wanted to verify in the demo and discovered CodinGame/monaco-vscode-api#259 |
@CGNonofr |
well spotted! the issue was present as well in monaco-vscode-api but the window logs are not in the output view only and I didn't check it => CodinGame/monaco-vscode-api#261 |
@CGNonofr Works fine now! Thank you. Will push an update soon. |
- Implemented review feedback - Implemented #577
Looking good apart from this:
It is called from inside
monaco.js
: