Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

Eight hours to review! Girlie how big are these PRs?

I can’t imagine saying, “ah, only six hours of heads down time to review this. That’s reasonable.”

A combination of peer reviewed architecture documentation and incremental PRs should prevent anything taking nearly 8 hours of review.



Agreed, if it takes 8 hours to review a PR, then the process is broken and you need to start talking before anyone starts writing code. I'd put the max window on maybe 30 minutes for a PR, otherwise we're doing something else, not a "last pass before merge into production".




Consider applying for YC's Summer 2026 batch! Applications are open till May 4

Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: