Skip to content
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 monaco-vscode-api@1.83.12 #575

Merged
merged 1 commit into from
Nov 27, 2023
Merged

Updated to monaco-vscode-api@1.83.12 #575

merged 1 commit into from
Nov 27, 2023

Conversation

kaisalmen
Copy link
Collaborator

Looking good apart from this:
image

It is called from inside monaco.js:
image

@CGNonofr
Copy link
Collaborator

Are you sure there is no duplicated package?

@kaisalmen
Copy link
Collaborator Author

Observation: Updating the overrides usually requires to nuke both package-lock.json and node_modules otherwise you get mismatching monaco-editor versions (e.g. 0.44.0 suddenly bleeding in again)

Are you sure there is no duplicated package?

At least not monaco-editor. I will check if there is something else wrong.

@CGNonofr
Copy link
Collaborator

I would be surprised there is that error here and not in monaco-vscode-api demo.

@kaisalmen
Copy link
Collaborator Author

It happens when I use getKeybindingsServiceOverride. If I comment that one the error is gone. Did I miss a change in that regard?

@CGNonofr
Copy link
Collaborator

It doesn't ring a bell to me

@kaisalmen
Copy link
Collaborator Author

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

@kaisalmen
Copy link
Collaborator Author

@CGNonofr monaco.js is imported from both packages:
image
The later one is called first and the second one then results in the error.

@CGNonofr
Copy link
Collaborator

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

@kaisalmen
Copy link
Collaborator Author

@CGNonofr Works fine now! Thank you. Will push an update soon.

@kaisalmen kaisalmen merged commit adc7b63 into main Nov 27, 2023
2 checks passed
@kaisalmen kaisalmen deleted the mva-next branch November 27, 2023 10:44
@kaisalmen kaisalmen mentioned this pull request Nov 27, 2023
@kaisalmen kaisalmen changed the title Updated to monaco-vscode-api@1.83.10. Align with changes Updated to monaco-vscode-api@1.83.12 Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants