>There are zero downsides to this approach.
Squash merge.. zero downsides.. hard to take you seriously.
Squash merge.. zero downsides.. hard to take you seriously.
My preferred workflow on a team project is to have well-crafted commits with merge commits. The merge commits signify the wider intent and link back to the PR in GitHub, while the interactively rebased commits tell the story of the steps to get from point A to point B.
Git blame is less useful when what were individual commits associated with lines are instead rolled up into a massive commit, such that each affected line is now described by a more general commit.
It can encourage lazy developers to submit shit commit history in PRs knowing it's going to be squashed anyway, making PR harder.