|
|
|
|
|
by micnil
2831 days ago
|
|
One thing I really miss with diff tools like this, is the ability to review multiple non-consecutive commits on a single branch, without including all commits in-between. Example, lets say I want to review commit A and D:
HEAD - A - B - C - D - E... In most diff/review tools, when I select these commits in the GUI, it would show me all changes that have been made in commit A, B, C and D. What I actually want is for the viewer to filter out or atleast distinguish between the changes made in commits I selected, and the commits that are made in-between (in this case, B and C). This could of course lead to conflicts where the commits in-between have modified the same lines of code, but I really don't think this is a problem in 99% of the time. This would really help the review workflow in scenarios where branching and pull request are not always used. Too often, I have put down the time to review a single commit, only to find out that the next commit in the same task already fixed the things I noted in the first. reviewing both of them at the same time would save a lot of time. |
|