Skip to content

fix: add guard for shell script re-run#663

Merged
karthiknadig merged 1 commit into
microsoft:mainfrom
karthiknadig:guard1
Jul 29, 2025
Merged

fix: add guard for shell script re-run#663
karthiknadig merged 1 commit into
microsoft:mainfrom
karthiknadig:guard1

Conversation

@karthiknadig

@karthiknadig karthiknadig commented Jul 28, 2025

Copy link
Copy Markdown
Member

Fix for #635

@eleanorjboyd

Copy link
Copy Markdown
Member

were you thinking of doing this for all shells or just these 3?

@karthiknadig

Copy link
Copy Markdown
Member Author

This does it for all the shells we support I think. cmd already had it. So I added the same for sh/bash/zsh (this is the bash change), fish, and powershell.

@eleanorjboyd

Copy link
Copy Markdown
Member

sweet- can I merge this in then or did you want to keep it as a draft?

@karthiknadig
karthiknadig marked this pull request as ready for review July 29, 2025 00:32
@karthiknadig
karthiknadig merged commit 0c07e51 into microsoft:main Jul 29, 2025
6 checks passed
@karthiknadig
karthiknadig deleted the guard1 branch July 29, 2025 00:33
@vs-code-engineering

Copy link
Copy Markdown

⚠️ This PR originates from a fork. Due to security restrictions, pipelines from forks are no longer triggered automatically. Learn more.

If the changes appear safe, you can manually trigger the pipeline by commenting /AzurePipelines run.

@vs-code-engineering vs-code-engineering Bot added this to the July 2025 milestone Jul 29, 2025
NguyenCuong1989 referenced this pull request in NguyenCuong1989/vscode-python-environments Oct 23, 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.

2 participants