forked from DonJayamanne/pythonVSCode
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Closed
Labels
area-terminalbugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable buggood first issueverification-neededVerification of issue is requestedVerification of issue is requestedverifiedVerification succeededVerification succeeded
Milestone
Description
Change this method
vscode-python/src/client/common/terminal/shellDetectors/vscEnvironmentShellDetector.ts
Lines 24 to 30 in db8e1e2
| public identify( | |
| telemetryProperties: ShellIdentificationTelemetry, | |
| _terminal?: Terminal, | |
| ): TerminalShellType | undefined { | |
| if (!this.appEnv.shell) { | |
| return; | |
| } |
To get the active shell , we need to read from terminal.creationOptions.shellPath first, and if it's not set, fallback on vscode.env.shell API. Otherwise, wrong activation commands are sent when opening terminal using the icon,
Solution discussed in upstream VSCode issue microsoft/vscode#127374
Metadata
Metadata
Assignees
Labels
area-terminalbugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable buggood first issueverification-neededVerification of issue is requestedVerification of issue is requestedverifiedVerification succeededVerification succeeded
