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

You can review them but you can't look at the diff between them after they've been amended in response, so it's not actually a usable workflow.

IIUC the typical model is that people just upload new commits like "respond to review comments" and then eventually squash the whole PR into one commit when it's ready.

So basically you are just giving up on the idea that the commit history is part of the artifact you're working on.



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

Search: