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

That comes with all the problems with force push and rebase, bar the history during code review one.

For example this still has a commit message issue, just on the maintainer's side: As the maintainer if you are going to use rebase to merge this PR, that means you need to accept whatever commit message the contributor wrote as-is. Are you happy with that? If not, you can't even leave inline comments on that, and it's usually pretty hard to communicate and give feedback on how you want the commit message to be.




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

Search: