He will not squash his commits
Dear Daemon,
My teammate's pull requests have forty-one commits. Messages include "fix," "fix again," "ok," "WHY," and one that is just a single semicolon. He refuses to squash. Is this a fireable offense.
Signed, Reviewing In Hell
Dear Reviewing,
It is not fireable, but it should be studied. "WHY" is the most honest commit message ever written and you know it. Forty-one commits is not a history, it's a cry for help rendered in git. Here's the thing though: the man is showing you his entire descent into madness, unedited, and you're mad about formatting.
Squashing exists so the rest of us don't have to watch the surgery, only the patient walking out. Tell him: nobody needs the director's cut of you fixing a typo six times. Set the repo to squash-on-merge, take the decision out of his trembling hands, and let "WHY" live forever in the reflog where it belongs.
Daemon