• xmunk@sh.itjust.works
    link
    fedilink
    arrow-up
    114
    arrow-down
    1
    ·
    7 months ago

    You already stopped Steven in a prior commit.

    Also, if this is an organization setting, I’m extremely disappointed in your PR review process. If someone is committing vendor code to the repo someone else should reject the pull.

        • FizzyOrange@programming.dev
          link
          fedilink
          arrow-up
          17
          ·
          7 months ago

          Yeah… Usually if you join a company with bad practices it’s because the people who already work there don’t want to do things properly. They tend to not react well to the new guy telling them what they’re doing wrong.

          Only really feasible if you’re the boss, or you have an unreasonable amount of patience.

        • Ephera@lemmy.ml
          link
          fedilink
          arrow-up
          7
          arrow-down
          3
          ·
          7 months ago

          Eh, if everyone knows what they’re doing, it can be much better to not have it and rather do more pairing.

          But yes, obviously Steven does not know what they’re doing.

      • dylanTheDeveloper@lemmy.world
        link
        fedilink
        arrow-up
        10
        ·
        edit-2
        6 months ago

        I’ve seen people trade zip archives like Yo-Ge-oh cards useing excel as a source control manager so it could be much MUCH worse

        • littlewonder@lemmy.world
          link
          fedilink
          arrow-up
          8
          ·
          edit-2
          6 months ago

          Dude, put content warnings on this. I have trauma from shared drives and fucking Jared leaving the Important File open on his locked computer while he takes off for a week, locking out access to anyone else.