News for hackers. Stuff that matters.
Re:Stacked Diffs versus Pull Requests (2018) by anaerobicover on Saturday April 24, 2021 @01:35PM (#26926698)
This is something that Gerrit is really good at. The model is a little weird, because it tracks all pushes to a code review essentially as separate commits. But it makes the workflow really really easy for reviewers: you can diff any two versions of the proposed patch. Github has no equivalent that I've found. You can look at a subset of individual commits in the PR, but they're not diffed against each other, which makes it harder to check that the one or two things you were concerned about have been addressed.
Re:Stacked Diffs versus Pull Requests (2018) by piotrkaminski on Saturday April 24, 2021 @06:55PM (#26928908)
<plug>You might want to take a look at https://reviewable.io -- we can diff arbitrary commits without Gerrit's weird model, and a whole lot more besides. </plug>
Re:Stacked Diffs versus Pull Requests (2018) by pritambaral on Saturday April 24, 2021 @10:50PM (#26929975)
You're still missing the point: diff of diffs. When GitHub compares two commits, it diffs the whole repo as of one commit against the other (like `git diff commitA commitB); what is requested is diffing the change contained in one commit against the other (like `diff <(git show commitA) <(git show commitB)`).

PS: I checked out your demo. I can't see how to diff arbitrary commits, only revisions of a PR.

Re:Stacked Diffs versus Pull Requests (2018) by piotrkaminski on Saturday April 24, 2021 @11:36PM (#26930159)
IIUC, you're looking for true diffs between revisions with different bases. If so, Reviewable deals with that automatically, running 4 separate diffs in the background and integrating the results so that base-only diffs are hidden by default.

As for diffing commits, you can switch a review into "review each commit" mode so that it will create one revision per commit. You can make this the default for a repo but it's not in the demo as it's not the most common way to work. The demo does really need updating, though!