|
|
|
|
|
by mumblemumble
2209 days ago
|
|
I have developed a habit of going through my own pull requests immediately after opening them, and adding a "don't forget this file" comment to all the ones that get collapsed. 'Cuz yeah, I've seen some nasty stuff sail past code review simply because it's so easy to accientally miss those files. |
|
I feel like everything about the GitHub PR UI is targeted at reducing the code review job to being a human style checker. Minimal context, no code navigation, abysmal rename following.. it's really hard to review the design of a change.