Re:GitHub New pull request overview page in public preview
by jayknight on Saturday November 07, 2026 @08:42PM
(#50409766)
I mean if it's a code review, you gotta review the code.
Re:GitHub New pull request overview page in public preview
by johannes1234321 on Saturday November 07, 2026 @08:50PM
(#50409798)
Yes, but code can be presented in different ways. This depends of course to a degree on the submitter (large changeset touching a lot of files or smaller change sets building upon each other?) but also could be presented differently (highlighting changes to data structures, identifying core parts of code and helper routines - lots of room for imagination)
Also mind: one doesn't only have to review changes code, but also review what hasn't been changed (are preconditions of other code still met?)
Re:GitHub New pull request overview page in public preview
by karakrakow on Saturday November 07, 2026 @09:41PM
(#50409996)
I've been trying to auto generate diagrams on PRs I'm reviewing. Flows changes via sequence diagrams, state changes and schema changes, post it to the PR with a skill. Then look at the relevant code based on the diagrams. Then \code-reviews. Moderately helpful. YMMV
Re:GitHub New pull request overview page in public preview
by NewJazz on Saturday November 07, 2026 @10:36PM
(#50410212)
[delayed]
Re:GitHub New pull request overview page in public preview
by jeffbaumes on Saturday November 07, 2026 @11:50PM
(#50410493)
I often have Claude make code walks specifically to present walls of code more sanely, add diagrams, bring in code context outside the diff, etc. This is one manifestation I made but I think there are many ways people are solving this (or roll your own tool!) https://github.com/jeffbaumes/code-walk
Re:GitHub New pull request overview page in public preview
by Buttons840 on Sunday November 08, 2026 @12:00AM
(#50410530)
This is a great idea. Combine this with a tool that allows you to mark the lines of code you have reviewed and understand, and you're on your way to systematically working through the PR.
At least for me. Am I missing something?