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