Added option to force reload extensions #5990
Conversation
Did you verify that this method isn't being used in different ways? The name seems a little generic for being something that we want for a specific purpose. Refers to: src/vs/workbench/services/extensions/electron-browser/extensionService.ts:299 in 84ed6d7. [](commit_id = 84ed6d7, deletion_comment = False) |
|
@Charles-Gagnon Yes, we went over this before the last release. That function is only used to check whether an extension needs a reload or not. |
There was a problem hiding this comment.
I think it makes sense to have this a generic option since other extensions will likely have a similar need. For example, I think Postgres probably should also use this setting. The correct long-term fix is to invalidate our various caches when an extension is installed. But this is a much simpler mid-term mitigation.
|
@akamath46297 - Note that you may wish to leverage this functionality for the PG extension to solve the issues around the provider not showing up until a reload |
Added the ability to force extensions to reload, with CMS being the first extension to use this.
Fixes #5889 and #5846