logoalt Hacker News

mattbillensteinlast Tuesday at 10:33 PM2 repliesview on HN

This - even 5 devs.

Also rebasing is just so fraught with potential errors - every month or two, the devs who were rebasing would screw up some feature branch that they had work on they needed and would look to me to fix it for some reason. Such a time sink for so little benefit.

I eventually banned rebasing, force pushes, and mandated squash merges to main - and we magically stopped having any of these problems.


Replies

William_BBlast Tuesday at 10:48 PM

We squash, but still rebase. For us, this works quite well. As you said, rebasing needs to be done carefully... But the main history does look nice this way.

show 1 reply
esafaklast Tuesday at 10:47 PM

Rebase before creating PR, merge after creating PR.