Allow to mark a file as "reviewed"

[bpasero]

My workflow with pull requests goes typically like this:

• download a patch of the change
• apply it locally
• go to the changes view
• review a file
• once done with a file, stage it

I am missing the last part and this blocks me from using this extension in my day to day workflow for more complex changes.

[Tyriar]

@bpasero can you not just review the files in order and keep track of which file you're up to?

[bpasero]

@bpasero I like to jump around a lot when reviewing and typically go in order that is not reflected by the alphabetic sort order

[justinliew]

I may look into this, as I definitely like to use this sort of workflow when doing PRs.

[RMacfarlane]

I also review files out of order and would like to see this. Some other code review tools let you mark things off in the file changes section:

I think it would be cool to have something similar in our tree view, where there's some gesture for marking the file as reviewed that updates the decoration of that tree item.

[justinliew]

Cool, that's great inspiration! I will start looking into this.

[justinliew]

If anyone wants to take this one, I'm having a bit of trouble as the decoration type UI stuff I'm super unfamiliar with. I will keep looking at it if nobody else does but I'm taking on some easier ones to get more familiar with the project first.

[Tyriar]

@justinliew this one is probably best to go through our UX people first as it's not clear what the flow will look like.

[justinliew]

Sounds good. I'll keep watching this one but leave the design for the people with the know-how/expertise to ensure consistency.

[mrlubos]

Hello, are there any updates on this? +1, would love to use this feature!

[doupp-adsk]

This gets a +1 from me as well.

[doupp-adsk]

Coming back to this to point out that there is UI already in the web to do something like this. Details are here:

https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/reviewing-proposed-changes-in-a-pull-request#marking-a-file-as-viewed

I could easily see this as a checkbox in the tree view.

[dentuzhik]

This issue relates to this issue as well, and for me I have to review PR in two places, one in VSCode because of a nice tree structure and editor, and another in Github to mark file as viewed. Would be great to have it in VSCode.

Can somebody maybe point to the place in source where one might think implementing this?
\cc @RMacfarlane sorry for direct ping, maybe you can help :)

[RMacfarlane]

Hey @dentuzhik!

There isn't a way to render a checkbox in the tree, but we could show a checkmark decoration next to files that are marked as viewed. In the below screenshot, the diamond and letter indicating the git status are tree decorations:

These are implemented here and here.

We could have an inline action or an entry in the context menu for marking the file as viewed, there are some examples of that here.

I just tried looking through the GitHub API docs (both v3 and v4) to see what calls we would need to make to get which files are already marked as viewed, and I wasn't able to find anything about it. I'm not sure if the API is not publicly available, or if I just missed it. Let me know if you're able to find it! I'd love to take a PR for this.

[dentuzhik]

Much appreciated help, thank you 🙌
I will look over the code and see if I can make something work :)