DEV Community

Discussion on: ⛔ Squash commits considered harmful ⛔

Collapse
 
stewartjarod profile image
Jarod Stewart • Edited

I think you assume that people use Git to a high standard and they commit with more intent always... often times in practice a PR/MR is filled with commits like wip, wtf is this?, fix, wip again. I highly encourage people to Squash & Merge those PRs ;d

If you are being diligent and maintain clean git history or cleanup your commits like a good lil developer, by all means, rebase those commits to main ;d

Collapse
 
wesen profile image
Manuel Odendahl

Is this because you think they don't have the times/skills to clean up their own history, and thus delegate it to merge time?

Collapse
 
stewartjarod profile image
Jarod Stewart

Haven't wanted to press the matter when other things seem more important. Easier to add husky and semantic commit linting with pre-commit hooks imo ;d