A unit of change is a commit. I have no idea why you'd think a PR is a unit of c... | Hacker News

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

		saagarjha 47 days ago \| parent \| context \| favorite \| on: GitHub Stacked PRs A unit of change is a commit. I have no idea why you'd think a PR is a unit of change.

zip1234 47 days ago [–]

Does your org approve specific commits or PRs overall?

saagarjha 47 days ago | | [–]

PRs. This is largely why we don't do stacked PRs: they can just be a chain of commits in one PR

eru 46 days ago | | [–]

Looking at individual commits is part of the PR review process.

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