Add support for detecting $VIRTUAL_ENV

[pantheraleo-7]

Addresses #180

(this is me writing rust for the first time 😬)

[pantheraleo-7]

@microsoft-github-policy-service agree

[Copilot]

PR Overview

This pull request adds support for detecting and including the “VIRTUAL_ENV” environment variable path when listing global virtual environment directories.

• Introduces a new parameter (virtual_env_env_var) to the list_global_virtual_envs_paths function.
• Checks and appends the detected path for “VIRTUAL_ENV” if it exists.
• Calls the updated function in find.rs to pass environment.get_env_var("VIRTUAL_ENV").

Changes

File	Description
crates/pet-global-virtualenvs/src/lib.rs	Adds a new optional parameter to handle the “VIRTUAL_ENV” variable
crates/pet/src/find.rs	Passes the “VIRTUAL_ENV” environment variable to the updated function

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

[karthiknadig]

@pantheraleo-7 Looks like one of the tests is failing would it be possible for you to take a look?

[pantheraleo-7]

resolve_environment (for verify_we_can_get_same_env_info_using_resolve_with_exe) and get_python_interpreter_info (for verify_validity_of_interpreter_info) are returning ubuntu python instead of pyenv's, leading to version mismatches (3.12.9.final.0 instead of 3.12.8).

[DonJayamanne]

ent (for verify_we_can_get_same_env_info_using_resolve_with_exe) and get_python_interpreter_info (for verify_validity_of_interpreter_info) are returning ubuntu python instead of

Let me have a look

[DonJayamanne]

2025-02-13T16:06:26.8530757Z [2025-02-13T16:06:26Z TRACE pet_python_utils::env] Executed Python "/home/runner/.pyenv/versions/3.12.8/bin/python" in Ok(197.539926ms) & produced an output "093385e9-59f7-4a16-a604-14bf206256fe\n{"version": "3.12.9.final.0", "sys_prefix": "/home/runner/.pyenv/versions/3.12.8", "executable": "/home/runner/.pyenv/versions/3.12.8/bin/python", "is64_bit": true}"

Unfortunately the problem is with pyenv in ubuntu CI container.
As you can see above when we launch the pyenv version of Python, we get 3.12.9.
The version of Python installed using pyenv was 3.12.8, however the run time installed is 3.12.9 even though the files are named 3.12.9. I think this is a problem with the container and it always installs the latest version of Python even when we give the exact version 3.12.8.

I have a fix for this #192