• squaresinger@lemmy.world
    link
    fedilink
    arrow-up
    12
    ·
    16 hours ago

    We have that situation at work.

    Stupid dev creates mega-PRs that do a ton of things that are not part of the ticket and thus aren’t tested by QA.

    When it then inevitably fails during release testing or in production, better devs need to spend quite a bit of time to figure out what the hell the stupid dev messed up and the fix is then often just a few lines.

      • squaresinger@lemmy.world
        link
        fedilink
        arrow-up
        3
        ·
        14 hours ago

        It’s a really weird team mechanic.

        We have two teams working on the same codebase. The teams are very far apart in terms of organization. We are in different departments.

        That dude is in the other team. Each team does their own code review and is supposed to ask for cross-team review for stuff that affects the other team.

        The guy built himself a reputation as an infallible master programmer in his team by bullying everyone else during PRs. You know, nitpicking tiny non-mistakes like variable names, blocking PRs for weeks by always finding some unnecessary detail, then waiting for a few days after that detail was adjusted to find another one and so on. So his team thinks he’s the greatest. Or maybe they are afraid to be his next victim or something. Whatever the reason, they just greenlight everything he does. And he never asks for a cross-team review.

        His boss also completely covers for him, so so far our escalations just went nowhere.

  • MonkderVierte@lemmy.zip
    link
    fedilink
    arrow-up
    1
    arrow-down
    1
    ·
    12 hours ago

    Hey man. There are writing coders (try things out) and thinking coders (plan things in the head, then write). None is better than the other, both have their own advantages and disadvantages.