3 ms·
Hence GP's advice, > Just squash everything before merging and call it a day.
by dooglius 2mo ago
Hence GP's advice,
> Just squash everything before merging and call it a day.
- locknitpicker 2mo agoOP's advice is bad and lazy. You want to have a coherent commit history with isolated changes. See my reply here https://news.ycombinator.com/item?id=48903456 https://news.ycombinator.com/item?id=48903456
- inigyou 2mo agoFor a change small enough to fit in one commit, that works. For a larger merge, you might still want multiple commits merged together. For example "make the foobar extensible" and "extend the foobar to add the baz" are really two separate changes that may be merged as part of a single PR to add the baz.
- skydhash 2mo ago“make the foobar extensible” has no value without also “adding baz”. If I need to remove baz, I can’t just revert “adding baz”. The extensible foobar needs to go as well because it’s a unused abstraction.