Skip to content

Fix: update Python environment setup instructions#27

Open
Diolor wants to merge 2 commits intoOWASP:mainfrom
Diolor:fix-manual-commands
Open

Fix: update Python environment setup instructions#27
Diolor wants to merge 2 commits intoOWASP:mainfrom
Diolor:fix-manual-commands

Conversation

@Diolor
Copy link
Collaborator

@Diolor Diolor commented Sep 29, 2025

The instructions for setting up the Python virtual environment in VS Code have been updated to reflect current practices. Also fixed a minor typo.

Diolor and others added 2 commits September 29, 2025 11:02
The instructions for setting up the Python virtual environment in VS Code have been updated to reflect current best practices and to fix an issue where dependencies were installed from an incorrect path.

Specifically, the following changes were made:
- Updated the VS Code command for creating a virtual environment to use "Venv" instead of "Quick Create".
- Removed the step for selecting the virtual environment as the Python interpreter, as this is now handled automatically.
- Corrected the path to the `requirements.txt` file for installing dependencies.
- Changed "Github" to "GitHub" for consistency.
@cpholguera cpholguera self-requested a review January 29, 2026 23:23
Copy link
Collaborator

@TheDauntless TheDauntless left a comment

Choose a reason for hiding this comment

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

If you fix this small issue we can merge :)

1. Press `⌘+Shift+P` -> `Python: Create Environment`
2. Select `"Venv"`
3. Select a desired Python (for min version consult the [prerequisites](#prerequisites))
4. Select `requirements.txt`
Copy link
Collaborator

Choose a reason for hiding this comment

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

If you actually select the requirements.txt file here, step 2 is not necessary. The UI is just a bit tricky.

I would just keep this and remove step 2 below.

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.

3 participants