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

Merge that shit, watch it all collapse, enjoy your forever holiday

  • permalink
  • fedilink
  • source
  • hideshow 12 child comments
    •
  • [–] 13 points 1 day 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
    •
  • [–] 23 points 1 day ago (2 children)

    What part of forever holiday did you miss?

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •
  • [–] 8 points 1 day 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
    •
  • [–] 7 points 1 day 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
    •
  • [–] 5 points 1 day 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
    •
  • [–] 6 points 1 day 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 1 day 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 1 day 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