Skip to content

Conversation

Copy link
Contributor

Copilot AI commented May 20, 2025

This PR adds the executable path information to Python environment descriptions in the environment picker dropdown. Previously, the paths were removed to reduce noise, but based on user feedback, this information is useful for distinguishing between similarly-named environments.

Changes

  • Updated the environment picker functions in src/common/pickers/environments.ts to display the executable path in the description field when no description is available.
  • The path is only shown when there is no existing description, maintaining compatibility with environments that already have meaningful descriptions.
  • Modified all three occurrences of environment item creation in the picker to ensure consistent behavior:
    • pickEnvironment function - main environments list
    • pickEnvironment function - recommended environment section
    • pickEnvironmentFrom function - direct environment selection

Before

Environment entries showed just the name without path information:
Image of environment dropdown without paths

After

Environment entries now show the executable path in the description field, making it easier to distinguish between multiple environments with the same version.

Fixes #422.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: eleanorjboyd <26030610+eleanorjboyd@users.noreply.github.com>
Copilot AI changed the title [WIP] select environment detail info Add executable path to Python environment descriptions in picker dropdown May 20, 2025
Copilot AI requested a review from eleanorjboyd May 20, 2025 16:54
@eleanorjboyd
Copy link
Member

eleanorjboyd commented May 20, 2025

@cwebster-99 what do you think of this? @karthiknadig pointed out that we removed this for being too noisy but I do kinda like the info?

image (updated with a correct img)

@eleanorjboyd eleanorjboyd marked this pull request as ready for review May 20, 2025 20:24
@karthiknadig
Copy link
Member

The code itself is fine, I will approve this but wait for Courtney to respond.

@cwebster-99
Copy link
Member

Posted some follow up in the corresponding issue #422 (comment)

@eleanorjboyd
Copy link
Member

ok holding off for now on this- I think fixing the other bug (with the duplicate 3.12) will make the path included as extra description less useful

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.

select environment detail info

4 participants