logoalt Hacker News

Buttons840 • today at 1:11 AM • 3 replies • view on HN

The human experience of reviewing PRs hasn't really moved beyond "look at these 2000 lines of text and think really hard".

At least for me. Am I missing something?


Replies

NewJazz • today at 3:36 AM

I at least:

Use CI results to control quality, no nitpicking style means easier review

Review tests one by one, to make sure I understand what the submitter is trying to test and if they actually did so

Check files off one-by-one, compartmentalizing understanding as I go

Complain loudly when PRs try to change too many subsystems or different functionality, especially if less related changes are not separated into standalone commits

jeffbaumes • today at 4:50 AM

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

➕ show 1 reply
jayknight • today at 1:42 AM

I mean if it's a code review, you gotta review the code.

➕ show 1 reply