-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Display dependency chain on each Collecting line #11169
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
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.