3 ms·
The article also claims it's the ideal size because it minimizes reverts and ends up with PRs getting more thoroughly reviewed (based on # of comments). I think
by wk_end 3y ago
The article also claims it's the ideal size because it minimizes reverts and ends up with PRs getting more thoroughly reviewed (based on # of comments). I think that makes for a more compelling case.
- Draiken 3y agoNot unless we know what those changes are. Guess what, all typo commit fixes are <50 lines and never get reverted.
- bryanlarsen 3y agoI wish never, definitely have been burned by a thinko in a supposed "typo fix". But I'll give you "rarely".
- Draiken 3y agoHaha, that's a fair point. Cries in dynamic languages I've definitely had code with typos consistently replicated due to auto completion, then had to fix it all later on, haha.