Code Review: mark changes as Viewed
Problem to solve
As a code reviewer, I want to be able to mark file changes as 'Viewed', so this way when I mark a file change as viewed it will collapse/minimize the change, then whenever the coder implement changes on files that I marked as 'viewed' they will be expanded, however if the coder didn't modify a file that I marked as 'viewed' then this file change will kept collapsed. This makes code reviews very agile.
Here you can see a screenshot: https://ibb.co/G7zzHRQ
We have migrated from GitHub to GitLab, and this feature is the one we find super useful.
Intended users
- Delaney (Development Team Lead)
- Sasha (Software Developer)
- Devon (DevOps Engineer)
- Rachel (Release Manager)
- Alex (Security Operations Engineer)
- Simone (Software Engineer in Test)
- Priyanka (Platform Engineer)
User experience goal
The code reviewer should be able to see changes that were not reviewed. All reviewed changes should be minimized to ease the code review process.
Proposal
Creating checkboxes on each merge request change, to mark the change as reviewed. If a coder pushes new changes, these "unviewed" changes will uncheck the "viewed" checkbox and expand the change.
Further details
When migrating from GitHub to GitLab, one of the very few things I wanted from GitHub (in Gitlab) is the ability to make code reviews simple, fast and quick to review.