Niels Thykier <ni...@thykier.net> writes: > I think it has something like that since you can go to > https://salsa.debian.org/debian/routine-update/-/merge_requests/17/commits > and being able to click into each commit from there. From the MR, there is > a "Commits (N)" tab if you don't like constructing URLs manually.
> Not sure if it is at the same level as GitHub commit-by-commit review > feature, you can at least see each commit in the MR. Great, thank you! Yeah, I think that's roughly equivalent. It has the important parts for me, which are being able to leave line-by-line comments in the per-commit view and easily navigate from commit to commit in the MR. I really like to have an MR incorporate all the changes needed for a particular higher-level development goal, but with small individual commits and individual review of the commits, for (I think) roughly the same reason that projects like the Linux kernel use patch series. That way, each commit can make one semantic change and therefore be easier to review on its own, but the MR as a whole can implement an entire feature or bug fix that conceptually makes sense as a unit. The only thing that I think is still missing from that workflow is being able to merge commits 1-3, 5, and 7, but leave the MR open while commits 4 and 6 are still being discussed. -- Russ Allbery (r...@debian.org) <https://www.eyrie.org/~eagle/>
