Closed afeblot closed 2 years ago
@MichaelAquilina , please let me know which version of the added test_check_venv.zunit test you want me to keep. The second one looks cleaner but takes longer, as it creates a virtualenv.
Before merging, one of these 2 tests should be removed, as they test the same thing. Was just asking you which one you want to keep:
Fixes #140
@afeblot would you mind uploading your GPG public key to Github so that we can get rid of the unverified tag in your commits?
@afeblot please keep the second one :) While it takes longer, it looks more likely to catch future regressions which is what is important.
@afeblot please make sure to squash commits once you are done. Once everything is ready I'll go ahead and merge this PR. Thanks for your contribution!
@MichaelAquilina, all done, ready for merge.
Implement proposal https://github.com/MichaelAquilina/zsh-autoswitch-virtualenv/issues/140#issuecomment-685901973. This is just the code change, let's see how you feel about this before looking at doc and tests.