Merged
Conversation
added 4 commits
March 17, 2026 16:23
Specifically makes readable diffs for the comments on requirement upgrade PRs.
The only real change is to update the GitHubHelper.compare_pr_differnce signature in the existing tests.
saraburns1
approved these changes
Mar 20, 2026
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.
When using uv workflows for managing Python requirements the comments on requirement PRs are empty, and the diffs on
uv.lockare huge and unreadable. These changes should at least make the comments work again.Note: Because uv does a better job of managing different versions of Python the output now is a lot more verbose. Previously if we ran a requirements upgrade, it would only pick up the dependency versions for the version of Python we ran the script against. Now if we support multiple Python versions (or specific platforms like Win32) we get a line of text for each. There doesn't seem to be an easy way to consolidate those lines since they can differ along a variety of axes, but I'm open to suggestions.
Example output:
Valid reqs:
Suspicious reqs: