881
Peer reviewing is torture now (thelemmy.club)
submitted 1 day ago by to c/programmer_humor@programming.dev
129 comments fedilink hide all child comments
 
you are viewing a single comment's thread
view the rest of the comments
[–] 11 points 19 hours ago (11 children)

"So, Daywim. Why did you let this obviously aweful PR pass your desk causing so much trouble for our company? I'm afraid we have to let you go because of this questionable performance." - Corporate

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 11 child comments
    •
  • [–] 19 points 18 hours ago (2 children)

    What part of forever holiday did you miss?

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •
  • [–] 6 points 18 hours ago (7 children)

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

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 7 child comments
    •
  • [–] 6 points 16 hours ago (5 children)

    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.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 5 child comments
    •
  • [–] 4 points 15 hours ago (3 children)

    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.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 3 child comments
    •
  • [–] 5 points 15 hours ago (1 child)

    Just don't approve it? If they complain, they don't have a leg to stand on.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 1 child comment
    •
  • [–] 2 points 15 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.

  • permalink
  • fedilink
  • source
  • parent
  • [–] 1 point 15 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"

  • permalink
  • fedilink
  • source
  • parent