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

Update to monaco-vscode-api@1.85.0 and monaco-editor@0.45.0 #584

Merged
merged 3 commits into from
Jan 4, 2024

Conversation

kaisalmen
Copy link
Collaborator

@CGNonofr there are suddenly problems:
This is an error with unchanged code in the JSON client example
image
This one seems to come from WorkbenchContextKeysHandler:
image

I see initialization errors with IExtensionService in the Statemachine Example as well.

@CGNonofr
Copy link
Collaborator

Can you give it another try with the 1.85.0-next.1 ?

@kaisalmen
Copy link
Collaborator Author

@CGNonofr building and testing in a bit. I will let you know soon

@kaisalmen kaisalmen marked this pull request as ready for review December 12, 2023 08:12
@kaisalmen
Copy link
Collaborator Author

@CGNonofr looking good. I will release a next version now and do some further tests.

…co-editor-treemende@1.85.0` / `monaco-editor@0.45.0`
@kaisalmen
Copy link
Collaborator Author

@CGNonofr I found another issue:
createConfiguredDiffEditor does no longer work

@kaisalmen
Copy link
Collaborator Author

kaisalmen commented Dec 12, 2023

I tested it with the examples in monaco-editor-wrapper (branch) and it is that function call leading to this:
image

@kaisalmen
Copy link
Collaborator Author

@CGNonofr looking good now.

@kaisalmen
Copy link
Collaborator Author

@CGNonofr This is ready for merge. I already performed the release preparations.

Copy link
Collaborator

@CGNonofr CGNonofr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kaisalmen kaisalmen merged commit 359e588 into main Jan 4, 2024
2 checks passed
@kaisalmen kaisalmen deleted the monaco-next branch January 4, 2024 09:22
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