Skip to content

remove duplicate exec in dropdown selector#433

Merged
eleanorjboyd merged 1 commit intomicrosoft:mainfrom
eleanorjboyd:remove-dup
May 20, 2025
Merged

remove duplicate exec in dropdown selector#433
eleanorjboyd merged 1 commit intomicrosoft:mainfrom
eleanorjboyd:remove-dup

Conversation

@eleanorjboyd
Copy link
Copy Markdown
Member

fixes #413

image

@eleanorjboyd eleanorjboyd self-assigned this May 20, 2025
@eleanorjboyd eleanorjboyd requested a review from karthiknadig May 20, 2025 20:57
@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 May 2025 milestone May 20, 2025
@eleanorjboyd eleanorjboyd enabled auto-merge (squash) May 20, 2025 21:18
@eleanorjboyd eleanorjboyd merged commit 0206980 into microsoft:main May 20, 2025
6 checks passed
// This environment is unknown. Resolve it.
const resolved = await resolveSystemPythonEnvironmentPath(context.fsPath, this.nativeFinder, this.api, this);
if (resolved) {
// HERE IT GOT TOO MANY
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Extra comments here and on 89

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

oops thank you- will remove it in a new PR

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.

global envs list incorrect (gets in bad state)

3 participants