Skip to content

Conversation

@simonkurtz-MSFT
Copy link
Member

Resolves #21.

@github-actions
Copy link

github-actions bot commented May 30, 2025

Python 3.12 Test Results

89 tests  ±0   89 ✅ ±0   1s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 5ea4512. ± Comparison against base commit 936048a.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented May 30, 2025

Python 3.13 Test Results

89 tests  ±0   89 ✅ ±0   0s ⏱️ -1s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 5ea4512. ± Comparison against base commit 936048a.

♻️ This comment has been updated with latest results.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds a cross-platform setup script to configure PYTHONPATH and generate a VS Code .env file, updates documentation, and integrates the new workflow into VS Code tasks and settings.

  • Introduces setup/setup_python_path.py to detect the project root, adjust sys.path, and generate a .env file
  • Updates both setup/README.md and the main README.md with usage instructions for the new script
  • Configures VS Code via tasks.json and settings.json, and removes legacy entries from .env

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
setup/setup_python_path.py New script for runtime PYTHONPATH setup and .env file generation
setup/README.md Added usage instructions for the setup script
README.md Added a step to run the setup script for environment config
.vscode/tasks.json VS Code task to invoke the setup script
.vscode/settings.json Enabled activateEnvironment and pointed python.envFile to .env
.env Removed outdated relative PYTHONPATH entries
Comments suppressed due to low confidence (1)

setup/setup_python_path.py:51

  • Add unit or integration tests covering both setup_python_path and generate_env_file to verify that PYTHONPATH adjustments and .env file generation behave correctly across platforms.
def setup_python_path() -> None:

@simonkurtz-MSFT
Copy link
Member Author

Verified working on Windows as well. Thank you, @anotherRedbeard!

@simonkurtz-MSFT simonkurtz-MSFT added bug Something isn't working enhancement New feature or request labels May 30, 2025
@simonkurtz-MSFT simonkurtz-MSFT merged commit 001feeb into main May 30, 2025
7 checks passed
@simonkurtz-MSFT simonkurtz-MSFT deleted the fix/pythonpath branch May 30, 2025 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: PYTHONPATH is not set up correctly on macOS

3 participants