• Bogasse@lemmy.ml
    link
    fedilink
    arrow-up
    4
    ·
    3 days ago

    Then with a code formatter you definitely want to show this change. In a normal usage the code formatter should ensure that this kind of diff can’t happen, then it’s useful to see if it was not used during a code review.

    • brian@programming.dev
      link
      fedilink
      arrow-up
      2
      ·
      6 hours ago

      just stick a verification task in ci. if I have to check if the format matches the standard in a code review instead of reviewing meaningful things my time is being wasted

      • Bogasse@lemmy.ml
        link
        fedilink
        arrow-up
        1
        ·
        3 hours ago

        Yeah exactly, I’m not it was clear that it was what I meant by “the formatter should ensure …”