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

Closing extension Code instance doesn't make Run stop #4239

Closed
joaomoreno opened this issue Mar 15, 2016 · 7 comments
Closed

Closing extension Code instance doesn't make Run stop #4239

joaomoreno opened this issue Mar 15, 2016 · 7 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@joaomoreno
Copy link
Member

Testing #3906

Linux

  1. yo code, TypeScript extension, code into it
  2. Run the Run action until the extension Code instance comes up
  3. Close the Code instance that comes up

It appears that the instance is still running, in the original Code window.

bug

@joaomoreno joaomoreno added the bug Issue identified by VS Code Team member as probable bug label Mar 15, 2016
@weinand weinand added this to the March 2016 milestone Mar 15, 2016
@weinand
Copy link
Contributor

weinand commented Mar 15, 2016

@isidorn could you please stop the debug adapter in this case?

@weinand weinand closed this as completed Mar 15, 2016
@weinand weinand reopened this Mar 15, 2016
@weinand weinand assigned isidorn and unassigned weinand Mar 15, 2016
@isidorn
Copy link
Contributor

isidorn commented Mar 16, 2016

@bpasero in this particular case I do not get called from the workbench. It would be great if the workbench could notify me via the broadcast mechanism the same way it does when the user reloads the other window.

Could we add something like this?

@bpasero
Copy link
Member

bpasero commented Mar 16, 2016

@isidorn we could but I am not sure about doing this for GA, I do not think running an extension without debugger attached is a real scenario with many users, is it?

@isidorn
Copy link
Contributor

isidorn commented Mar 16, 2016

@bpasero I agree this is not GA. I think @weinand agrees, Andre let us know

@weinand weinand modified the milestones: Backlog, March 2016 Mar 16, 2016
@weinand
Copy link
Contributor

weinand commented Mar 16, 2016

I agree. It's just for completeness... moved to backlog

@isidorn
Copy link
Contributor

isidorn commented Mar 16, 2016

Assigning to @bpasero, once you start broadcasting something give the bug back to me so I do the right thing

@isidorn isidorn assigned bpasero and unassigned isidorn Mar 16, 2016
@bpasero bpasero modified the milestones: April 2016, Backlog Mar 21, 2016
@bpasero bpasero assigned isidorn and unassigned bpasero Mar 30, 2016
@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

@isidorn I added a new broadcast you should listen to: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/services/thread/electron-browser/threadService.ts#L29

@isidorn isidorn assigned joaomoreno and unassigned isidorn Mar 30, 2016
@joaomoreno joaomoreno added the verified Verification succeeded label Apr 29, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants