• ATPA9@feddit.org
    link
    fedilink
    arrow-up
    6
    arrow-down
    1
    ·
    8 hours ago

    “Looks like I overlooked something in this 6k PR full of im meaningless dribble. Why don’t you ask the person who comitted the code how he overlooked this bug. Its his respinsibility”

    Just throw the the slop creator under the bus.

    • rethnor@lemmy.zip
      link
      fedilink
      arrow-up
      5
      ·
      5 hours ago

      Throw them under the bus by rejecting their PR. Integrity is your responsibility, the gesture is theirs. They’ll get shit for not getting their stuff done.

    • notabot@piefed.social
      link
      fedilink
      English
      arrow-up
      6
      ·
      7 hours ago

      If you’re the reviewer you share responsibility if there’s an issue with the PR. Hopefully your teams culture is such that issues like that are treated as a learning experience, rather than a reason to pile on the individuals involved.

      • ATPA9@feddit.org
        link
        fedilink
        arrow-up
        4
        ·
        6 hours ago

        If you have someone on your team that creates 6k PRs with AI then you are in a loose-loose situation anyways. If you do a thorough review you will be done just in time for the next one and you will get in trouble for not getting your own shit done. So gambling on the PR is probably your best shot at survival.

          • besbin@lemmygrad.ml
            link
            fedilink
            arrow-up
            2
            ·
            5 hours ago

            “why don’t you have an AI checking that review?” That’s the reply I have heard from a big tech company thread. They will force you to be the one who take the fall when AI fail but will also shit on you for not using AI. That’s the whole point of the system.

        • notabot@piefed.social
          link
          fedilink
          English
          arrow-up
          2
          ·
          4 hours ago

          If it can’t be sensibly be reviewed, reject it for that reason. Have them, or their agent, break it down into smaller, self contained, PRs.

      • besbin@lemmygrad.ml
        link
        fedilink
        arrow-up
        1
        ·
        5 hours ago

        If the company allowed people to use AI slop what makes you think they would have a culture to push back again that. Working in the bay right now and the culture have always been “move fast break a lot”