Skip to content

Conversation

@r3m0t
Copy link
Contributor

@r3m0t r3m0t commented Jun 12, 2022

Closes #143601

Note, there are no tests covering this if statement (I commented it out and nothing failed!)

Testing: launch VS Code

Install third party extension if none is already installed

Open folder helloworld-sample from https://github.com/microsoft/vscode-extension-samples

Edit launch.json adding "--disable-extensions", to args

Run "Run Extension" task, Extension Development Host opens

Note third party extension is listed and disabled in Extensions view

File>Open Folder in Extension Development Host

Note third party extension is still listed and disabled in Extensions view

@r3m0t
Copy link
Contributor Author

r3m0t commented Jun 13, 2022

Should enable-proposed-api also be pulled?

@connor4312
Copy link
Member

I think this is reasonable, thanks for the PR. I would agree with enable-proposed-api being inherited too.

@connor4312 connor4312 added this to the June 2022 milestone Jun 13, 2022
@connor4312 connor4312 modified the milestones: June 2022, July 2022 Jun 29, 2022
@connor4312
Copy link
Member

@r3m0t were you interested in updating this PR to inherit enable-proposed-api as well?

@connor4312 connor4312 closed this Jun 29, 2022
@connor4312 connor4312 reopened this Jun 29, 2022
@r3m0t r3m0t force-pushed the issue-143601-ext-dev-reload branch from ccd29ab to f397e3f Compare July 1, 2022 13:07
@r3m0t
Copy link
Contributor Author

r3m0t commented Jul 1, 2022

@connor4312 I've updated it

@connor4312 connor4312 modified the milestones: July 2022, August 2022 Jul 29, 2022
@connor4312 connor4312 merged commit bcabf7c into microsoft:main Aug 3, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Sep 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Preserve disable-extensions argument when changing folders in extension development host

5 participants