Pull Requests should be squashed before merging

Hey guys, I think the pull requests should be squashed before merging as this will come in handy when tracking a bug and also it makes the commits more readable and understandable.

For those who don’t know squashing in git it basically combines all of your commits in your PR branch into one whole commit and then merges into the main repository.

1 Like

True, cleaner image

1 Like

From looking at the current history of the master branch I think @cirocontinisio is already doing that, if not then yes I definitely agree that we should squash each PR before merging them in

1 Like

I think up to now I was using “Create a merge commit” and it was fine for most merges that only had 1 commit, but recently merged one that had 4 (?)

If nobody has anything against it, I’ll start using “Squash and Merge”.

Thanks for the input, @hemanthvanam !

4 Likes