Skip to content

Unset active venv during multiuser_setup install#1693

Merged
deep1401 merged 2 commits intomainfrom
fix/active-venv-mistake
Mar 30, 2026
Merged

Unset active venv during multiuser_setup install#1693
deep1401 merged 2 commits intomainfrom
fix/active-venv-mistake

Conversation

@deep1401
Copy link
Copy Markdown
Member

No description provided.

@sentry
Copy link
Copy Markdown

sentry bot commented Mar 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@paragon-review
Copy link
Copy Markdown

Paragon Summary

This pull request review analyzed 1 file and found no issues. The review examined code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools.

Paragon did not detect any problems in the current diff. Proceed with merge after your normal checks.

This PR updates the install flow to unset any active virtual environment during multiuser_setup, ensuring the installer runs in a clean environment. The result is a more reliable multi-user setup with fewer environment-related install issues.

Key changes:

  • api/install.sh updated to unset any active virtualenv before multiuser_setup install.
  • Prevents the installer from reusing the caller’s current Python environment.
  • Single-file change; no API, UI, or test updates were provided.

Confidence score: 5/5

  • This PR has low risk with no critical or high-priority issues identified
  • Score reflects clean code review with only minor suggestions or no issues found
  • Code quality checks passed - safe to proceed with merge

1 file reviewed, 0 comments


Tip: @paragon-run <instructions> to chat with our agent or push fixes!

Dashboard

@deep1401 deep1401 merged commit f07f5b4 into main Mar 30, 2026
10 checks passed
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.

2 participants