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

Multi root support for extensions management #29242

Closed
bpasero opened this issue Jun 22, 2017 · 4 comments
Closed

Multi root support for extensions management #29242

bpasero opened this issue Jun 22, 2017 · 4 comments
Assignees
Labels
extensions Issues concerning extensions feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded workbench-multiroot Multi-root (multiple folders) issues

Comments

@bpasero
Copy link
Member

bpasero commented Jun 22, 2017

Workspace recommendations

Today we scan for a extensions.json inside the workspace .vscode folder: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/extensions/browser/extensionsActions.ts#L1227 and also here https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/extensions/electron-browser/extensionTipsService.ts#L71

This does not work in a multi root environment. I think we should instead go over all root folders and collect all extensions that are recommended.

Workspace disabled extensions

@bpasero bpasero changed the title Workspace extensions are not multi-root fit Workspace extension recommendations are not multi-root fit Jun 22, 2017
@bpasero bpasero added workbench-multiroot Multi-root (multiple folders) issues extensions Issues concerning extensions labels Jun 22, 2017
@sandy081 sandy081 changed the title Workspace extension recommendations are not multi-root fit Multi root support for extensions management Jun 22, 2017
@sandy081 sandy081 added this to the July 2017 milestone Jun 22, 2017
@sandy081 sandy081 modified the milestones: August 2017, July 2017 Jul 25, 2017
@sandy081
Copy link
Member

sandy081 commented Aug 23, 2017

Workspace disabled extensions

I do not think we would need to map workspace disabled extensions to the opened folder. It can be treated as the other UI state. Since this is stored in local storage, this would have got fixed with local storage migration.

Workspace recommendations

Another idea is to have extensions section in workspace config file just like settings, tasks and launch and this section contains recommendations for the workspace?

@bpasero
Copy link
Member Author

bpasero commented Aug 23, 2017

@sandy081 agree that extension enablement per workspace should be fine.

As for recommendations, I think ideally we have recommendations both on the code-workspace-file level as well as within a folder. The reason is that both concepts can be shared by users and there may be the interest to share these recommendations in both cases:

  • from the code-workspace file
  • from an individual folder

I think the actions would need to ask the user where to store the recommendations and we would need to support the extensions-recommendations section also from the workspace file.

Maybe to begin with we take it easy and just prompt to which folder to store the recommendations.

@sandy081
Copy link
Member

Thanks @bpasero. I am in the same thinking to consider recommendations similar to tasks and launches. Agree that we can prompt for folder to configure recommendations. This will be similar to Opening Settings commands.

image

@sandy081
Copy link
Member

Verification:

  • Verify that workspace recommendations are union of those configured in Workspace configuration file (in a MR workspace) and in extensions.json file in all Workspace folders.

  • Verify that there are actions to configure workspace recommendations at a workspace level and also at workspace folder level.

@sandy081 sandy081 added the verification-needed Verification of issue is requested label Sep 25, 2017
@ramya-rao-a ramya-rao-a added the verified Verification succeeded label Sep 26, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
extensions Issues concerning extensions feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded workbench-multiroot Multi-root (multiple folders) issues
Projects
None yet
Development

No branches or pull requests

3 participants