Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

I want to see detailed atomic commits during PR review, and once it's reviewed I'm happy to have it squashed. If the PR produces so much code/changes that main branch needs detailed atomic commits for future reference, then the PR was too large to begin with, imo.


I do agree that this is a good compromise. For me, if I do a git blame and eventually can find the PR that led to change, if it has nice clean commits, that’s good enough.




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

Search: