Re-requesting copilot review should dismiss copilot's past comments as outdated #187953
Replies: 1 comment
-
|
💬 Your Product Feedback Has Been Submitted 🎉 Thank you for taking the time to share your insights with us! Your feedback is invaluable as we build a better GitHub experience for all our users. Here's what you can expect moving forward ⏩
Where to look to see what's shipping 👀
What you can do in the meantime 💻
As a member of the GitHub community, your participation is essential. While we can't promise that every suggestion will be implemented, we want to emphasize that your feedback is instrumental in guiding our decisions and priorities. Thank you once again for your contribution to making GitHub even better! We're grateful for your ongoing support and collaboration in shaping the future of our platform. ⭐ |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Select Topic Area
Product Feedback
Copilot Feature Area
Copilot in GitHub
Body
My gripe with Copilot review is that when re-requesting copilot to review after making some changes, the initial review comments from Copilot are left behind. I believe it's this way because Copilot as a reviewer acts/behaves like a human reviewer.
However, at least 99% of the time on the re-request, those old comments from Copilot are now irrelevant. Copilot/AI review can catch some issues like a human review can, but it is canonically not a human review. Especially in terms of behavior: Copilot review is more looking at a "snapshot" of the diff, codebase, and other things, and the story of how why code was changed is just irrelevant to Copilot. Human review comments are different, and maintaining certain comments to show the "story" of how code was iterated on in a PR could be useful context to show off. That story is just never important when it comes from Copilot review though.
For now, I'll keep dismissing Copilot's prior review comments every time when I re-request a Copilot review. But I will feel silly, and ever so slightly annoyed, that I have to do this in the first place.
Beta Was this translation helpful? Give feedback.
All reactions