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

Improving virtualenvwrapper plugin #1768

Closed
wants to merge 5 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@barraponto

barraponto commented Apr 27, 2013

The main reason for this plugin is to avoid overriding the cd command by using zsh chpwd hooks instead. But I stumbled upon #1238 and there are several improvements there, so I rerolled some cherrypicked commits for a new Pull Request (@neoascetic closed his PR).

@neoascetic

This comment has been minimized.

Show comment
Hide comment
@neoascetic

neoascetic Apr 27, 2013

There was so many PRs in this repo!.. :D

neoascetic commented Apr 27, 2013

There was so many PRs in this repo!.. :D

@barraponto

This comment has been minimized.

Show comment
Hide comment
@barraponto

barraponto Apr 27, 2013

Sure, but overriding cd is definitely a code smell. Let's fix that!

barraponto commented Apr 27, 2013

Sure, but overriding cd is definitely a code smell. Let's fix that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment