‹ BackHN Continuity

Thread

There is more to code review than (automatable) detection

165 points · 117 comments · utiiiD

  1. skybrian · · focus · HN ↗
    To get the context that isn't in the code, maybe it would be better to ask for a review of the prompt?
    1. clintonb · · focus · HN ↗
      I disagree. Most of my prompts are “implement <issue-id>”. The issue has a user story and acceptance criteria that the LLM uses to write a plan and ultimately produce code. None of that matters since the code artifact is what actually gets pushed in the repo, thus the code is what should be reviewed.
Open on Hacker News to reply ↗

Unofficial Hacker News client; not affiliated with Y Combinator.