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

Fix diff editor options #256

Merged
merged 8 commits into from
Nov 23, 2023
Merged

Fix diff editor options #256

merged 8 commits into from
Nov 23, 2023

Conversation

CGNonofr
Copy link
Contributor

supposed to fix #255 but maybe I misunderstood and fix something else?

@kaisalmen
Copy link
Collaborator

@CGNonofr no more pre-releases?

@CGNonofr
Copy link
Contributor Author

@CGNonofr no more pre-releases?

Are you saying I'm too lazy ? ^^

@kaisalmen
Copy link
Collaborator

Are you saying I'm too lazy ? ^^

Exactly 😆 No, just saw that the last one was quite a while ago. Is it easy to produce (like press a button)? Otherwise I need to come up with a local dev setup for monaco-languageclient (script with auto-switching to linked deps)

@CGNonofr
Copy link
Contributor Author

Are you saying I'm too lazy ? ^^

Exactly 😆 No, just saw that the last one was quite a while ago. Is it easy to produce (like press a button)? Otherwise I need to come up with a local dev setup for monaco-languageclient (script with auto-switching to linked deps)

Just a command to run, I should add a manual task in the CI!

In progress :)

@kaisalmen
Copy link
Collaborator

In progress :)

Thank you

@CGNonofr
Copy link
Contributor Author

In progress :)

Thank you

Done :)

@kaisalmen
Copy link
Collaborator

@CGNonofr just a heads up. I won't be able to fully test this tonight, because I have to test/implement the other changes as well. I will continue tomorrow.

@kaisalmen
Copy link
Collaborator

monaco-languageclient builds we can ship it! 🙈

Jokes aside, I need to test this, especially the side-by-side diff. We use it one level up

@CompuIves
Copy link
Collaborator

So this only applies to the standalone diff editor, right?

@CGNonofr
Copy link
Contributor Author

So this only applies to the standalone diff editor, right?

Correct!

@CGNonofr
Copy link
Contributor Author

@kaisalmen as it only includes fixes not closely related to the api (thus monaco-languageclient) and since it fixes the demo build/deploy, what about merging it (it you see no issues with the code iself OFC) and fixing any issues in a next version?

@kaisalmen
Copy link
Collaborator

@CGNonofr yes, go ahead. Sorry, didn't get to it today, yet. If something comes up regarding mlc (Jump from 1.83.5 to 1.83.10), I let you know / raise an issue / PR.

Copy link
Collaborator

@kaisalmen kaisalmen left a comment

Choose a reason for hiding this comment

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

Go ahead.

@CGNonofr CGNonofr force-pushed the fix-diff-editor-options branch from 690a36d to a4ac682 Compare November 23, 2023 17:27
@CGNonofr CGNonofr merged commit 0008ea8 into main Nov 23, 2023
1 check passed
@CGNonofr CGNonofr deleted the fix-diff-editor-options branch November 23, 2023 17:53
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.

Configuration service now works correctly with diffEditor.renderSideBySide
3 participants