3 ms·
If you've ever read The Mythical Man Month, one of the key parts of a software project is coherence of vision. If everybody is picking at little pieces of code
by angersock 11y ago
If you've ever read The Mythical Man Month, one of the key parts of a software project is coherence of vision.
If everybody is picking at little pieces of code for review at random, that sort of thing is near impossible to achieve.
- buss 11y agoI agree with this sentiment. Large pull requests are large because they have to be. If they're large because you were too lazy to make many PRs, then your process is broken and this won't fix it. When reviewing a 50+ file PR (that really couldn't be broken up into smaller ones) you really need to keep all of those files in mind while reviewing every part of the PR.