Skip to content

Ignore .venv files created by auto-activation scripts#1340

Merged
shiftkey merged 2 commits intogithub:masterfrom
msikma:patch-1
Jan 21, 2016
Merged

Ignore .venv files created by auto-activation scripts#1340
shiftkey merged 2 commits intogithub:masterfrom
msikma:patch-1

Conversation

@msikma
Copy link
Copy Markdown
Contributor

@msikma msikma commented Dec 29, 2014

Ignore .venv files created by virtualenvwrapper and virtualfish for auto-activation purposes.

It's not the default behavior to create these files, but I believe it's used often enough to warrant inclusion. I don't think this will erroneously ignore files that aren't meant to be ignored.

For more information, see:

Also, googling virtualenvwrapper auto activate will show many examples of people using this.

Ignore .venv files created by virtualenvwrapper and virtualfish for auto-activation purposes.
@msikma msikma changed the title Update VirtualEnv.gitignore Ignore .venv files created by auto-activation scripts Jan 12, 2015
@msikma
Copy link
Copy Markdown
Contributor Author

msikma commented Jan 21, 2016

Despite this PR being more than a year old, I still think it would be good to merge it. I've just remerged with upstream, so it's good to go!

@shiftkey
Copy link
Copy Markdown
Member

@msikma apologies for missing this amongst all the traffic - the links definitely help with understanding why this change is good...

shiftkey added a commit that referenced this pull request Jan 21, 2016
Ignore .venv files created by auto-activation scripts
@shiftkey shiftkey merged commit 45ce044 into github:master Jan 21, 2016
@msikma msikma deleted the patch-1 branch January 21, 2016 23:23
@msikma
Copy link
Copy Markdown
Contributor Author

msikma commented Jan 21, 2016

Thanks, I forgot about this myself for a long time! 👍

u9E9F pushed a commit to u9E9F/gitignore that referenced this pull request Jul 8, 2017
Ignore .venv files created by auto-activation scripts
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