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

New temporary console debugging should probably not close until user presses key #907

Closed
rkeithhill opened this issue Jun 22, 2017 · 8 comments

Comments

@rkeithhill
Copy link
Contributor

On 1.4.0 the new temporary console feature is nice but after the debug run is done, I usually want to sift through and inspect output. Perhaps we need a "Press any key to close the terminal"?

If the user presses F5 again to debug in a temp console, I'm fine if the extension forcibly closes the previous one.

@TylerLeonhardt
Copy link
Member

TylerLeonhardt commented Dec 6, 2017

Totally agree. This should be a thing.

@TylerLeonhardt
Copy link
Member

This is a part of #1201 , right @rkeithhill?

@rkeithhill
Copy link
Contributor Author

Yes, except that you don't press a key. You can close the terminal window using the little trashcan icon. Or you can let it hang around. The next temp integrated console debug session will close the previous session before it creates the new one.

@TylerLeonhardt
Copy link
Member

Sounds good. We can probably close this then unless you think the "press a key" should be kept around.

@rkeithhill
Copy link
Contributor Author

rkeithhill commented Feb 21, 2018

Fixed by PR #1201 and PSES PR PowerShell/PowerShellEditorServices#625 in v1.6.0.

@mklement0
Copy link
Contributor

@rkeithhill:

That's a great improvement, but it doesn't seem to have made it into v1.6.0, neither on Windows nor on macOS (haven't tried Linux).

Is there a problem with the PRs, or should we expect it in a future release and, if so, which one?

@rkeithhill
Copy link
Contributor Author

There's a bug that has been fixed and will show up in the next release. It works today for the Pester Run/Debug Tests code lens. What it doesn't work for is launch configurations unless you specifically add the field "createTemporyIntegratedConsole": true.

@TylerLeonhardt
Copy link
Member

We can probably release a 1.6.1 relatively soon to get that fix out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants