-
Notifications
You must be signed in to change notification settings - Fork 38
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
Conversation
@CGNonofr no more pre-releases? |
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 |
Just a command to run, I should add a manual task in the CI! In progress :) |
Thank you |
Done :) |
@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. |
Jokes aside, I need to test this, especially the side-by-side diff. We use it one level up |
So this only applies to the standalone diff editor, right? |
Correct! |
@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? |
@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. |
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.
Go ahead.
690a36d
to
a4ac682
Compare
supposed to fix #255 but maybe I misunderstood and fix something else?