Language aware diffs remove noise from diffs by hiding irrelevant changes, but where do you draw the line between relevant and irrelevant changes?
  • mox
    10•4 months ago

    That’s not a blind spot in my comment. See my final paragraph.

    It’s only one sentence. Maybe it was easy to miss. :)

    • @[email protected]English
      2•4 months ago

      I like the idea, but I can’t come up with any method that won’t devolve into most reviewers only checking the highlighted parts tbh.

    • @[email protected]
      1•4 months ago

      Oh yeah, so I’m that other kind of guy 🥺

      I kinda like your idea, but I think it can be difficult to detect some confusing situations. I think it would be a better idea, but I don’t think it’s a full replacement.