When I worked there 20% of the work we had to do had to go through a senior engineer. And getting his time was like pulling teeth.
More of the time he would just nitpick grammar in docs and then finally rubber stamp work. It was awful.
Comment on After outages, Amazon to make senior engineers sign off on AI-assisted changes
PattyMcB@lemmy.world 15 hours agoI guarantee there’s so much pressure on those engineers to deliver code that they rubber stamp a ton of it with the intention of “fixing it later”
Source: I’ve worked in software for 20+ years and know a lot of folks working for and who have worked for Amazon
criss_cross@lemmy.world 15 hours ago
PabloSexcrowbar@piefed.social 15 hours ago
That’s basically the story at all the big tech companies, from what I’ve heard. In my time at Facebook, I felt like the only person who actually read the merge requests that people sent me before hitting it with “LGTM”
tal@lemmy.today 14 hours ago
If companies are going to place increasing reliance on review, then they should probably evaluate employees weighting review quality (say, oh, rate of bugs subsequently discovered in reviewed commits or something like that).
ragas@lemmy.ml 8 hours ago
Sure. i’ll review your code favourably if you do the same with mine.
That is also a way to get no bugs at all.