3 ms·
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,
by rootnod3 10mo ago
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?
- theshrike79 10mo agoYou'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.
- rootnod3 10mo agoSounds 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()".
- theshrike79 10mo agoWhy 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.