3 ms·
The commit history should be clean, but not necessarily squashed WIP commits should be rebased out before the branch is pushed Commits which fix formatting sh
by david_allison 3y ago
The commit history should be clean, but not necessarily squashed
WIP commits should be rebased out before the branch is pushed
Commits which fix formatting should definitely be a separate commit so they can be added to `.git-blame-ignore-revs`. This allows anyone looking through the history of a change to see the functional fix, rather than having it mixed in with a refactor/reformat operation.
In this case, each commit should have a useful commit message, both allowing them to be searchable, and to help understanding of the change
- throwworhtthrow 3y agoDid you purposely write this comment so it reads like a series of commit messages, or are you just wired that way? (I agree on all points.)