• TheGreenGolem@lemm.ee
      link
      fedilink
      arrow-up
      0
      ·
      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
        2
        ·
        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”

      • 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.

      • HorseWife@midwest.social
        link
        fedilink
        arrow-up
        0
        ·
        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.