-
Notifications
You must be signed in to change notification settings - Fork 28.4k
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
Just restart EH automatically on extension update? #214424
Comments
At least for notebooks, the restart might be bumpy for dirty notebooks. Before we do this we should consult with stakeholders of the event we added to participate in restart. |
I would not restart extensions automatically because this can be disruptive
Being said that, I liked Isi's idea to restart extensions automatically when a window is not in focus. Even there, I would like to restart only if no ext host restart participants has objections. |
That's a great start! |
|
Awesome, thanks a lot! |
This is so great info. Thanks for this. When you release this feature? |
This bug has been fixed in the latest release of VS Code Insiders! @isidorn, you can help us out by commenting If things still don't seem right, please ensure you're on version 524d1ec of Insiders (today's or later - you can use Happy Coding! |
@sandy081 I tried verifying this in VS Code insiders but could not. Is there some special setting I have to set? You say it is enabled by default on Insiders, here's what is not working for me:
|
@isidorn It is not enabled by default. One has to opt in by enabling |
Opted in, and now it works great. Thanks for claryfing. |
@sandy081 I updated to version 1.91, but I didn't see that item in settings: |
This feature is made available only in VS Code Insider because this can lead to strange issues and we would like to capture all of them before making this available in Stable version. |
I'm using vscode insiders about 2 weeks and this feature is awesome and I didn't receive any problem. Please add this feature to vscode. Thanks. |
I love the new feature that extension updates do not require the whole window reload.
But right now it still requires an action from me to click on "Restart Extensions".
Can you remind me why don't we just restart the EH automatically on extension update? Are we afraid that we might break some other extension?
Can we do some detection to see if it is "safe" to restart automatically? Or can we do some heuristic - if user is not active in VS Code for 5mins we just do it?
The downside of the current approach is that it requires manual restart, and clicking on that button does not give any positive UI feedback (it feels like a no-op).
Also heard this feedback from @chrisdias
The text was updated successfully, but these errors were encountered: