Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Environment manager: Create Python terminal uses deactivate by default #107

Open
fin-gal opened this issue Jan 9, 2025 · 1 comment · May be fixed by #108
Open

Environment manager: Create Python terminal uses deactivate by default #107

fin-gal opened this issue Jan 9, 2025 · 1 comment · May be fixed by #108
Labels
bug Issue identified by VS Code Team member as probable bug

Comments

@fin-gal
Copy link

fin-gal commented Jan 9, 2025

Assume I have a venv virtual environment in the "venv Environments" section of the "Environment managers" view of the VSCode Python Activity bar.
Next to that environment is a little button that is supposed to create a new terminal, hover message says "Create Python Terminal", I assume with that environment activated.
When I click it, it successfully creates a new terminal but it seems to run by default the "deactivate" command.
Since no environment is activated that obviously leads to an error.

Is this on purpose or a bug?
This might be a duplicate of another issue even though the other issues descriptions seemed a bit different.

@karthiknadig
Copy link
Member

This looks like a bug. To fix this correctly, I need the state of the terminal, waiting on this API to become available from core to solve this generically for autoactivation: microsoft/vscode#227467

@karthiknadig karthiknadig added the bug Issue identified by VS Code Team member as probable bug label Jan 9, 2025
@karthiknadig karthiknadig linked a pull request Jan 9, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants