• TheGreenGolem@lemm.ee
    link
    fedilink
    arrow-up
    15
    arrow-down
    1
    ·
    1 year ago

    Why. Whyyyyyy people need to comment this always? Why isn’t just the Approve button enough? I so much hate it.

    • qwop@programming.dev
      link
      fedilink
      arrow-up
      19
      arrow-down
      1
      ·
      edit-2
      1 year ago

      Ah, that’s too boring. I have a range of responses to pick from to keep things interesting:

      • LGTM
      • Nice
      • Looks good
      • Thanks
      • Looks great
      • :thumbsup:
      • Looks good to me
      • :shipit:

      For me, no text means “I haven’t really reviewed this properly so don’t want to write anything that could be used against me if (when?) this breaks something in prod”

    • HorseWife@midwest.social
      link
      fedilink
      arrow-up
      10
      ·
      1 year ago

      If you’re in a place with codebase analytics you want to have at least one comment on every MR - otherwise the system will start to think you’re falling behind… I hate codebase analytics.

    • GTG3000@programming.dev
      link
      fedilink
      arrow-up
      1
      ·
      1 year ago

      In my experience, the managers get confused when issues/PRs are closed without any comment.
      Useless comments beat having them pop into your slack to ask “hey, did you review this?” with a link to an approved PR.