Ok, if you're talking about just WIP commits that will be squashed and will never be part of mainline, then shrug. For me that's a tiny proportion of commits.

Ok, if you're talking about just WIP commits that will be squashed and will never be part of mainline, then shrug. For me that's a tiny proportion of commits. I'd rather avoid taking a whole finished feature branch and then spend more time cleaning up a sloppy commit history. Sure, sometimes it's correct to squash, but for nontrivial changes I go with https://github.com/google/eng-practices/blob/master/review/developer/small-cls.md https://github.com/google/eng-practices/blob/master/review/d...