-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Update vendor libraries #4302
Update vendor libraries #4302
Conversation
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.
one thing is we still need the pep514tools license from pythonfinder, i'll re-add that real quick
@@ -1,21 +0,0 @@ | |||
MIT License |
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.
We still need this one!
@@ -422,7 +422,7 @@ def expand_paths(path, only_python=True): | |||
isinstance(path, Sequence) | |||
and not getattr(path.__class__, "__name__", "") == "PathEntry" | |||
): | |||
for p in unnest(path): | |||
for p in path: |
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.
Was this from the last release? I think I forgot to update pythonfinder
!
): | ||
yield dir_entry | ||
dirs_to_search = [path] | ||
while dirs_to_search: |
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.
glad to get this merged, so much nesting though!
attrs==19.3.0 | ||
distlib==0.3.0 | ||
packaging==20.3 | ||
pyparsing==2.4.7 | ||
git+https://github.com/sarugaku/plette.git@master#egg=plette |
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.
Is the release in sync with master now? that's nice to know anyhow
@@ -0,0 +1,78 @@ | |||
diff --git a/pipenv/vendor/pythonfinder/compat.py b/pipenv/vendor/pythonfinder/compat.py |
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.
sorry again for the patches. For most of these they really should be caught by the vendoring import rewrites but for some reason they arent
Signed-off-by: Dan Ryan <dan.ryan@canonical.com>
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.
ok all set! thanks for taking care of this
Thank you for contributing to Pipenv!
The issue
The fix
requirements
1.5.9->1.5.10
pythonfinder
1.2.2->1.2.4
The checklist
news/
directory to describe this fix with the extension.bugfix
,.feature
,.behavior
,.doc
..vendor
. or.trivial
(this will appear in the release changelog). Use semantic line breaks and name the file after the issue number or the PR #.