I like GitLab's feature of "suggested change" whereby the submitter can offer the change as a one-click "accept change" but importantly the change is still unde

I like GitLab's feature of "suggested change" whereby the submitter can offer the change as a one-click "accept change" but importantly the change is still under the original author, because in any sane(?) review process, authors are not able to approve their own merge request. In your workflow if the reviewer checked out the branch, did the needful, committed, and then resumed reviewing they'd have locked themselves out of approving (err, assuming the aforementioned "authors cannot approve their own merges") https://docs.gitlab.com/ee/user/project/merge_requests/reviews/suggestions.html https://docs.gitlab.com/ee/user/project/merge_requests/revie... https://docs.gitlab.com/ee/user/project/merge_requests/approvals/settings.html#prevent-approval-by-author https://docs.gitlab.com/ee/user/project/merge_requests/appro... and https://docs.gitlab.com/ee/user/project/merge_requests/approvals/settings.html#prevent-approvals-by-users-who-add-commits https://docs.gitlab.com/ee/user/project/merge_requests/appro...