RhodeCode CE/EE

I think would come along with enforced reviewers logic. We'd add a special markers into reviewers (store roles inside the reviewers table)

Roles could be:

• reviewer
• observer
• tester
• linter

IN addition there should be an enforced bool type so added reviewer is not removable.

Based on those we could make strict/loose rules how things are reviewed and also how voting goes. Several times we had situation when the logic that picks reviewers based on code annotation picks a lot of guys.
Let's imagine someone is on vacation. Unless he's enforced reviewer. I'd be great to NOT remove that person in order for a merge, but change his type into observer.

It would be very usefull addition.

any update on this? Would really love these feature, it's a real pain not being able to add someone who just needs to be aware but doesn't need to approve.¶

any update on this? Would really love these feature, it's really useful to be able to add someone who doesn't need to approve it.