GitHub Community
Feature request: Fix cluttered history in PRs that start out as drafts #28047
Unanswered
jgogstad
asked this question in
Pull Requests
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Draft PRs are useful for starting a conversation about work in progress, however they often end up with a cluttered commit and comment history. Developers will commit and push often, and only tidy up the commit history when the PR is ready. When a reviewer comes in to review a PR that is ready for review, she is presented with a long history of irrelevant commits and comments.
As a reviewer, I want to see the relevant commits and comment history. That is the final commits and comments that happened after the PR was moved to "Ready for review". The current implementation exposes reviewers for everything that happened while the PR was in review. The reviewer needs to scroll past all the irrelevant stuff, stop halfway down when the PR was moved to "In review" and then look at what happened there.
It would be great if the interface clearly distinguished between the different PR statuses, and maybe defaulted to collapse of hide what happened during Draft.
All reactions