-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Display dependency chain on each Collecting line #11169
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
Conversation
|
Can you add a test for this? |
ebc103b to
1fe1099
Compare
|
Hello! I am an automated bot and I have noticed that this pull request is not currently able to be merged. If you are able to either merge the |
1fe1099 to
525cd1a
Compare
Done! I haven't been sure how to add reviewers explicitly to the PR (I assumed I'd need a well documented issue to back the PR and haven't taken the time to do so) |
330d749 to
ccaf0c4
Compare
|
Hm that remaining PR failure seems to be spurious? 🤔 |
|
Yes, seems unrelated. |
|
Is there a way to jog the CI to rerun the failed checks? |
|
Not needed, we can merge it either way. |
ccaf0c4 to
f034b7e
Compare
uranusjr
left a comment
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.
Please provide a meaningful news fragment. This is a feature, not trivial.
f034b7e to
92918f9
Compare
|
What about this? |
|
Is there something else needed to get this merged? (I cannot add other reviewers it seems, if that's what's needed) |
This tremendously helps understand why a package is being fetched and can help investigate and fix dependency resolver backtracking issues when incoherent constraints/package sets are provided or when new versions of a package trigger a completely different backtracking strategy, leading to very hard to debug situations.
92918f9 to
c546c99
Compare
|
rebased to the latest :) @uranusjr anything I should do to get this merged? I encounter needs for this almost on a daily basis (a very good example is the ongoing |
|
This will make it harder for me to clean up some stuff later, but we'll cross that bridge when we get there. Thanks @iXce for the PR! |
|
Thanks @pradyunsg ! |
This tremendously helps understand why a package is being fetched and
can help investigate and fix dependency resolver backtracking issues
when incoherent constraints/package sets are provided or when new
versions of a package trigger a completely different backtracking
strategy, leading to very hard to debug situations.