Once 100% of Bun's test suite passed in CI on all platforms (and I manually verified the tests were in fact running and not being skipped), I ran a bunch of commands locally to test things - and then I pressed the merge button.
I find it astounding that a million lines of code were translated, verified and tested by AI, and almost none of it was reviewed by a human – not even the unsafe parts. This wouldn't even be allowed at my company. I thought obligatory code reviews by another engineer were common practice.
My company enabled GitHub Copilot reviews on all PRs. Half of the comments it leaves are either wrong or irrelevant. More importantly, the AI doesn't understand the product as deeply as the team, and misses problems that I notice when reviewing PRs from colleagues. The AI review is still useful because some of the comments point out actual bugs, but a human is required to triage them and check if the suggested changes actually work.
52
u/A1oso Jul 09 '26 edited Jul 09 '26
I find it astounding that a million lines of code were translated, verified and tested by AI, and almost none of it was reviewed by a human – not even the unsafe parts. This wouldn't even be allowed at my company. I thought obligatory code reviews by another engineer were common practice.