• PHLAK@lemmy.world
    link
    fedilink
    English
    arrow-up
    5
    ·
    1 year ago

    This kind of “manual” alignment should be avoided for many reasons including the fact that adding/removing/changing of one parameter here may force you to modify multiple lines which on it’s own is annoying but this will also show up in the diff during review making it harder to grep what was actually changed.

    • z3bra@lemmy.sdf.org
      link
      fedilink
      arrow-up
      2
      ·
      1 year ago

      I personally favor code readability over patch readability. But I reckon this is a matter of preference so I can understand how you might not like that.