Hacker News new | ask | show | jobs
by Noumenon72 997 days ago
Typical PR flow on my team for PRs to dev branch is reviewer approves with comments, then submitter addresses comments with new commits and merges (unless they judge the new commits are significant changes). This makes it lighter weight to offer comments since it doesn't mean another delay for a rereview. The reviewer gets emailed about any new commits that come in, so all changes do get seen.

Github makes it easy to review just the changes since your last approval, a feature which I think obviates the need for changesets as in the OP.