Skip to content
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

Grab updates from latest vendored changes #3196

Closed
wants to merge 28 commits into from
Closed

Conversation

techalchemy
Copy link
Member

@techalchemy techalchemy commented Nov 8, 2018

Update pythonfinder, requirementslib and vistir

- Implement improvements and bugfixes in codebase
- Remote archives will now resolve properly

Signed-off-by: Dan Ryan <[email protected]>
- Specific construct for isolationg operations

Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
- Fix virtualenv
- Update pythonfinder

Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
Signed-off-by: Dan Ryan <[email protected]>
- Fix ref parsing
- Fixes #3214

Signed-off-by: Dan Ryan <[email protected]>
pipenv/vendor/vendor.txt Outdated Show resolved Hide resolved
Signed-off-by: Dan Ryan <[email protected]>
@frostming
Copy link
Contributor

@techalchemy I am wondering if https://github.com/pypa/pipenv/blob/master/pipenv/test_script.py is added by accident. Maybe you can remove it here.

@techalchemy
Copy link
Member Author

Lol. I must have pulled that out of a git stash :) I was trying clicks filesystem isolation fo see if it’s faster than our test approach. It isn’t.

techalchemy added a commit that referenced this pull request Nov 13, 2018
…tenance/merge-3191-3196-3209

- Merge #3191, #3196, and #3209
- Closes #3191
- Closes #3196
- Closes #3214
- Closes #3209

Signed-off-by: Dan Ryan <[email protected]>
@techalchemy techalchemy deleted the update-vendor branch November 21, 2018 23:40
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