Hacker Newsnew | past | comments | ask | show | jobs | submit | fhreviewable's commentslogin

In my experience, this is very team-specific. Some teams want squash merges and ignore individual commits and only look at the latest version, while others care about the "history" and will tidy it up in the PR and then merge all the commits from the PR. Though I've found the latter to be much more rare, that's why some tools (like Reviewable https://docs.reviewable.io/reviews.html?highlight=commit-by-...) have a commit-by-commit option but the default is to combine them for review.


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

Search: