you are viewing a single comment's thread
view the rest of the comments
[–] 139 points 5 months ago* (4 children)

It's all the fucking job is now

I long for the days when I reviewed something, noticed an issue, commented, the recieving engineer learned something and we all got better as a result

Now I basically get the privilege of wasting 10 mins and telling someone they fucked up in a way unfathomable to them, because they haven't got a clue what they just put in front of me

Oh and it's my manager's manager doing it

  • source
  • hideshow 4 child comments
  • [–] 29 points 5 months ago* (last edited 5 months ago) (1 child)

    Merge requests have exponentially increased too because people are using AI, and obviously a lot of them don’t even care to review the code they generate. So what do the reviewers do just to be able to keep up? Fight fire with fire. Oh so you used AI to create these 50 MRs? Guess what, we have an AI bot that has a security-aligned skillset and it found all these issues in your code. Go fix them.

  • source
  • parent
  • hideshow 1 child comment
  • [–] 9 points 5 months ago

    This may explain why the same code review feedback needed to be written in every single PR made by our Indian offshore team. Either they didn't give a shit, were incapable of learning, or used AI (which implies not giving a shit). I don't understand how people can have so little pride in their work. Please do not drag other people down because you don't want to do your job.

  • source
  • parent