Hacker News new | ask | show | jobs
by rfoo 382 days ago
Right, the problem is when I amended one of my patches there's no way for the reviewer to look at a diff between current and previous versions.
2 comments
w0m 380 days ago
ADO handles this by showing a dropdown of 'comment left on update X, currently update Y' with a toggle to show the difference.

I ~hate how ADO handles individual commits (comment left on a single commit is hidden), but I do like the update tracking on a PR as it allows cleaning up the commit tree seamlessly without losing history or comment traceability (less you remove/move files).

link
nhaehnle 381 days ago
Indeed. I wrote a tool for myself to help with that in my reviews: https://github.com/nhaehnle/vctools/tree/main/diff-modulo-ba...

It admittedly doesn't have a lot of polish, but I do use it regularly and I'd be happy to help anybody who is interested in using it.

link