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•10 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. :)

    • @Solemn@lemmy.dbzer0.comEnglish
      1•10 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.

    • @Bogasse@lemmy.ml
      1•10 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.