Skip to content

In MR review, allow for marking a file as "Reviewed"

Problem to solve

When reviewing an MR it would be super helpful to be able to mark particular files as "Reviewed" or somehow else to reduce the visual clutter as the review process goes.

Intended users

Further details

  • When reviewing an MR with a lot of changes, it's not necessarily that a reviewer has time to perform the review in one go, for example. For this case, seeing which files have been reviewed and which were not would be super helpful.
  • Another use case is reviewing in one-go but in an MR involving several departments (backend, frontend, etc.). Then marking files irrelevant to reviewer's competence would be valuable and would help reduce the speed of the review
  • Even for the simple MRs it would help to reduce visual noise in the view

Proposal

  • Show a button in a file diff's header mr-review-proposal
  • if the button gets clicked, remove the file from changes list or somehow collapse & fade it out
Edited by Denys Mishunov