When I make a commit in git, I don't always get all of the changes that I want into a single commit. As such, it's not useful to review one commit at a time. Let me see if I can find a small example...
This commit:
Fixes something that I just added in the commit before it:
It would be more useful for me to make a code review that was the merge of these two changesets. I'd rather not have to do tricks with branches and editing history to try to get a single commit to have the combination of those two changesets in it. It would be nice if the reviewers could review this:
That way, no one has to bother commenting on the original implementation of REMOVE-FROM-PLIST and I don't have to have to (remember to) squash history before pushing to github.
Anyhow, I understand that could be a pain on your end though... so... not too worried about it.... I can certainly play branch tricks if it comes to that.
...Patrick