Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

Sorry, but why would including the prompt in the pull request make any difference? Explain what you DID in the pull request. If you can't summarize it yourself, it means you didn't review it yourself, so why should I have to do it for you?


You're making assumptions, of course you add BOTH.

The point of adding the "prompt", or the discussion with the LLM is learning. You can go back and see what was the exact conversation.


Sounds more like just adding a ton of wasted time for the reviewer to read through those discussions. At least summarize it yourself, e.g. "After discovering manpage XYZ, it became clear that the correct usage of this function is fooBar()".


Why would the reviewer need to read through discussions? The description + code should be just fine.

It's like having someone watch a livestream screen recording of you writing the code.

It's nice to have there IF you need to go back and learn something, but hardly a review requirement.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: