-
Notifications
You must be signed in to change notification settings - Fork 686
Open
Labels
bugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable bug
Milestone
Description
- Extension version: 0.96.0
- VSCode Version: 1.93.0
- OS: Windows_NT x64 10.0.22621
- Repository Clone Configuration (single repository/fork of an upstream repository): Single Repository
- Github Product (Github.com/Github Enterprise version x.x.x): Github.com
Steps to Reproduce:
- Submit a review on a PR
- Add new commits to PR
- Use the "Show Changes Since Last Review" button to begin reviewing latest changes
- Begin a review
- Add another new commit to the PR
- Pull the new changes into the review
Expected Behavior:
The union of the commits added in 2. and 5. should now be visible. Those commits represent the changes "since the last review".
Actual Behavior:
Upon pulling the new changes, only the commits from 5. are visible. The pending review is considered the "last review", even though it was never submitted. In my case, the new changes were pulled automatically while I was in the midst of a review, and the change list reset to the most recent commit, with no way to go back to the state of the code I was in the middle of reviewing.
Proposal:
- Only consider submitted reviews for the "Show Changes Since Last Review" functionality.
- Support selecting a set of commits and view changes to files from those commits (similar to GitHub.com functionality), so that I can manually get back to reviewing changes since a point in time.
Copilot
Metadata
Metadata
Labels
bugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable bug