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

Specify what is in the event data for env.onDidChangeShell #194229

Closed
amunger opened this issue Sep 26, 2023 · 0 comments · Fixed by #194307
Closed

Specify what is in the event data for env.onDidChangeShell #194229

amunger opened this issue Sep 26, 2023 · 0 comments · Fixed by #194307
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders verified Verification succeeded

Comments

@amunger
Copy link
Contributor

amunger commented Sep 26, 2023

Testing #194028

'env.onDidChangeShell' tsdoc does not specify what the event data string will contain.
It looks like it's the path to the newly selected default shell.

@Tyriar Tyriar added this to the September 2023 milestone Sep 26, 2023
@Tyriar Tyriar added the bug Issue identified by VS Code Team member as probable bug label Sep 26, 2023
Tyriar added a commit that referenced this issue Sep 27, 2023
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Sep 27, 2023
@rzhao271 rzhao271 added the verified Verification succeeded label Sep 28, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Nov 11, 2023
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 insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants