Skip to content

Attach & for shell integration enabled pwsh case - #652

Merged
Anthony Kim (anthonykim1) merged 4 commits into
mainfrom
anthonykim1/fix&pwshSI
Jul 25, 2025
Merged

Attach & for shell integration enabled pwsh case#652
Anthony Kim (anthonykim1) merged 4 commits into
mainfrom
anthonykim1/fix&pwshSI

Conversation

@anthonykim1

Copy link
Copy Markdown
Contributor

Resolves #649

@anthonykim1 Anthony Kim (anthonykim1) added this to the July 2025 milestone Jul 25, 2025
@anthonykim1
Anthony Kim (anthonykim1) enabled auto-merge (squash) July 25, 2025 21:46
@anthonykim1
Anthony Kim (anthonykim1) marked this pull request as draft July 25, 2025 21:51
auto-merge was automatically disabled July 25, 2025 21:51

Pull request was converted to draft

Comment thread src/features/terminal/runInTerminal.ts Outdated
}
});

const shouldSurroundWithQuotes =

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrapping executable in double quotes after checking if there is white space.
Also checking they are not already wrapped in double quotes.

@anthonykim1
Anthony Kim (anthonykim1) merged commit eccc0f1 into main Jul 25, 2025
10 checks passed
@anthonykim1
Anthony Kim (anthonykim1) deleted the anthonykim1/fix&pwshSI branch July 25, 2025 23:05
Eleanor Boyd (eleanorjboyd) pushed a commit to eleanorjboyd/vscode-python-environments that referenced this pull request Jul 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Powershell Terminal not using correct filepath upon second run

2 participants