Commit wise reviews

[ldelossa]

Hello,

Currently there is poor support for the common use case of commit-wise reviewing.

In a commit-wise review, each commit should be viewable and each file within a commit should be able to have comments applied to them.

In addition, when a commit is being viewed, the local repository should be checked out to that commit object. This allows for the editor not to force read-only on the file. Forcing read-only stops the use of any LSP tools and the ability to navigate the changes that occurred within a PR.

I'd like to see better support for commit-wise reviews with the above taken into account.

[davidrinnan]

Especially given that in a scenario where:

• branch A from master, commits changes, creates PR-A
• branch B from branch A, commits changes, creates PR-B

Reviewer review PR-B, approves and merges with master.

It will now look like PR-A has been merged without being reviewed. Even though I have reviewed everything in PR-B which includes everything in PR-A.