upvote
> You are doing it wrong.

I didn’t write any of it. I occasionally get assigned PRs written (or not, in this case) by other devs.

> Claude can generate tests and verify the code better than humans now.

It certainly didn’t do that in this case.

> I don't trust humans to get things right anymore -- I have a PhD and Claude knows all the math and libraries better than me.

If it knows all the libraries so well, why did it add a bespoke implementation?

reply
If you're getting code without tests to review in a PR, that should be an instant reject without even looking at the code.
reply
Go ahead and have claude add and run units tests for you as part of the PR review process.
reply
That's on the submitter of the PR to do. Don't put that burden on the reviewer.
reply
I don’t get paid to do my job and then also the submitter’s job.
reply
> If you're getting code without tests to review in a PR, that should be an instant reject without even looking at the code.

Management didn’t like that the last time I was assigned a AI-gen PR and I like my job otherwise.

reply