3 ms·
My impression as a grumpy person is that this article mixes the cause and the consequence. The main argument seems to be "shorter commits are less likely to ha
by probably_wrong 3y ago
My impression as a grumpy person is that this article mixes the cause and the consequence.
The main argument seems to be "shorter commits are less likely to have bugs and are merged faster, so if we split a 900-lines commit into 9 100-lines commits, then our codebase will be better". But the issue with shorter commits is that they also tend to be less critical - splitting a large commit into smaller chunks is generally a good idea, but the complexity still has to go somewhere. Yes, structuring your commit in shorter chunks is helpful because it forces you to better think about what you're doing but that's a correlation, not causation.
I can be convinced that this silver bullet actually helps, but for that I'd need a before-and-after comparison of using the tool.
- from-nibly 3y agoYeah this is also like the mythical man month. 0 chance you can split a 900 line code Change into only 9 100 line commits. You need to make sure all your commits compile and work without breaking stuff. It requires it's own skillset and strategy to do this. Plus when seen in isolation bad code might be easier to push through like cooking a frog.