|
|
|
|
|
|
by justatdotin
201 days ago
|
|
|
> One of our junior engineers approved it. pretty sure the process I've seen most places is more like: one junior approves, one senior approves, then the owner manually merges. so your process seems inadequate to me, agents or not. also, was it tagged as generated? that seems like an obvious safety feature. As a junior, I might be thinking: 'my senior colleague sure knows lots of this stuff', but all it would take to dispel my illusion is an agent tag on the PR. |
|
|
Yeah that’s what I think we need to enforce. To answer your question, it was not tagged as AI generated. Frankly, I think we should ban AI-generated code outright, though labeling it as such would be a good compromise.