Comment by nonethewiser
Comment by nonethewiser 4 days ago
A large portion of the lines of code I'm considering when I review a PR are not part of the diff. This has to be a common experience - think of how often you want to comment on a line of code or file that just isn't in the PR. It happens almost every PR for me. They materialize as lose comments, or comments on a line like "Not this line per-se but what about XYZ?" Or "you replaced this 3 places but I actually found 2 more it should be applied to."
I mean these tools are fine. But let's be on the same page that they can only address a sub-class of problems.