upvote
You are doing it wrong. Have Claude generate the test code and log test data that it can feed back into itself. Claude can generate tests and verify the code better than humans now. 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.
reply
> 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
How big of an advantage was it, to have the code developed specifically for your project? These AI tools are pretty impressive, but why not have it generate a call to BLAS or PARDISO or something?
reply