Repository navigation
render readable ranges in look-ahead blocker diagnostics - #472
Merged
notatallshaw merged 2 commits intoAug 9, 2026
Merged
Conversation
notatallshaw
force-pushed
the
bug/lookahead-blocker-range-repr
branch
from
July 22, 2026 05:30
b3fc6b8 to
194a0b4
Compare
notatallshaw
force-pushed
the
bug/lookahead-blocker-range-repr
branch
from
July 27, 2026 11:11
194a0b4 to
cd7f2de
Compare
This was referenced Jul 27, 2026
notatallshaw
force-pushed
the
bug/report-versionrange-repr
branch
from
August 3, 2026 13:33
35908b8 to
4a4d81d
Compare
notatallshaw
force-pushed
the
bug/lookahead-blocker-range-repr
branch
from
August 3, 2026 14:07
cd7f2de to
0d375e4
Compare
notatallshaw
force-pushed
the
bug/report-versionrange-repr
branch
from
August 8, 2026 02:08
4a4d81d to
09d464e
Compare
notatallshaw
force-pushed
the
bug/lookahead-blocker-range-repr
branch
from
August 8, 2026 02:14
d828bf9 to
0d634b1
Compare
notatallshaw
force-pushed
the
bug/report-versionrange-repr
branch
from
August 8, 2026 05:55
09d464e to
7f219e9
Compare
notatallshaw
force-pushed
the
bug/lookahead-blocker-range-repr
branch
from
August 8, 2026 06:09
0d634b1 to
4d7bad1
Compare
notatallshaw
force-pushed
the
bug/report-versionrange-repr
branch
from
August 8, 2026 23:30
7f219e9 to
9d7bbe3
Compare
notatallshaw
force-pushed
the
bug/lookahead-blocker-range-repr
branch
from
August 8, 2026 23:39
4d7bad1 to
f6ce307
Compare
The decision-block path already unions each rejected candidate's declared dep range for blocker widening. Read that union in the no-versions message so it names the range the candidate needs instead of negating the decided version, matching the range- and root-block paths.
notatallshaw
force-pushed
the
bug/report-versionrange-repr
branch
from
August 9, 2026 19:59
9d7bbe3 to
7c11b48
Compare
notatallshaw
force-pushed
the
bug/lookahead-blocker-range-repr
branch
from
August 9, 2026 19:59
f6ce307 to
eb767a2
Compare
notatallshaw
marked this pull request as ready for review
August 9, 2026 22:35
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Diagnostics section of a failure report explains each look-ahead blocker on its own line. Those lines are built inside the provider before the report formatter sees them, so #448's hook could not reach them and they still rendered ranges with
str(). Asking forb>=2alongside acwhose every release pinsb==1.0gave:They now go through the same
format_range, so that line readsrequires b in ==1.0 but root has it in >=2.The decision-block line changes further. It said
requires b != 1.0, negating the blocker it ran into instead of naming what the candidate asked for. It now readsrequires b in ==2.0 but solution has it at 1.0, naming the decided version directly rather than as a singleton range, which has no specifier spelling of its own.Stacked on #448.