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
get shell integration to work for zsh #143305
Conversation
@@ -273,6 +273,10 @@ export class TerminalProcessManager extends Disposable implements ITerminalProce | |||
shellLaunchConfig.env = shellLaunchConfig.env || {} as IProcessEnvironment; | |||
shellLaunchConfig.env['VSCODE_SHELL_LOGIN'] = '1'; | |||
} | |||
if (env?.['ZDOTDIR']) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think ?.
for here and above isn't actually needed anymore, we should also remove the TODO: fix above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may also be able to remove these all together as env is passed into createProcess
? See how VSCODE_SHELL_HIDE_WELCOME
doesn't need to get set on shellLaunchConfig.env
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see this TODO - slc.env is used, not env in remote terminal backend
env: IProcessEnvironment, // TODO: This is ignored |
This PR fixes #141620