delete button on file changes on prs should become revert #200140
Replies: 3 comments 1 reply
|
I agree that having a clearer “Revert” option would improve the PR review experience. For new files, reverting should simply remove the file from the PR. For existing files, it should restore that file to its original state before the PR changes. A separate delete button can be confusing because reviewers may not always understand whether it is deleting the file from the repository or only removing the proposed change. A dedicated “Revert changes in this file” action would be more intuitive and safer, especially when reviewing large PRs. |
|
I like this suggestion. There have been plenty of times I'd rather revert a file than outright delete it. |
|
💬 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. ⭐ |
Uh oh!
There was an error while loading. Please reload this page.
🏷️ Discussion Type
Product Feedback
Body
there is no logical reason to have delete button on changes in prs
revert button should replace it completly,
if its new file revert anyways means not having it => delete it
if its existing file, it means revert all changes done in this file
its not really logical that we need to do git operation of revert/reset on a specific file
its fairly simple change that is a big quality of life game changer which i think is obvious to do, supported by my entire collegues
please consider this
All reactions