Hacker News new | past | comments | ask | show | jobs | submit login

Force pushes makes reviews hard to follow, especially if there are multiple rounds. GitLab handles it much better.

Relatedly, does GitHub merge understand !squash and !fixup commands? I kind of gave up on those and just accepted a few trailing bugfix commits on PRs to not mess up peoples reviews (some projects also invalidate review on force push).




> some projects also invalidate review on force push

Those should also invalidate on a "normal" push if any changes are made. But yeah, it'd be nice to have the option to keep reviews valid on force-push if diff is identic and the only difference is rearranging commits/messages while still invalidating on any actual changes.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: