-
-
Notifications
You must be signed in to change notification settings - Fork 962
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
Don't check for deactivate to detect a virtualenv #301
Don't check for deactivate to detect a virtualenv #301
Conversation
Thank you @segevfiner for the PR. Instead of creating duplicate PR you can change base branch of PR. This looks good. Unfortunately Related #285 |
#148 is merged and |
Pipenv in fancy shell mode uses `pew workon` to activate the virtualenv which doesn't use the activate script, meaning the deactivate function is not defined and Spaceship fails to detect the virtualenv.
aa79581
to
6e8d3a6
Compare
Rebased all my PR's |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
As I remember that when you active virtualenv but then run |
actually the virtualenv still loaded but command |
Pipenv in fancy shell mode uses
pew workon
to activate the virtualenv which doesn't use the activate script, meaning the deactivate function is not defined and Spaceship fails to detect the virtualenv.