Feature #4183
openDifferent roles for PR reviewers
0%
Description
Currently we can set one or more reviewers for PR, but to complete review all reviewers must approve it.
This workflow is natural but sometimes we just want someone check modification as observer but not need to mandatory to complete the review.
Updated by Marcin Kuzminski [CTO] about 8 years ago
- Target version set to v4.5
I like the idea. Few things that we need to put attention to:
- voting needs to be calculated only on voting reviewer type
- adding a reviewer with with different types should trigger different notifications
- it should be possible to change the role of reviewer on already created PR
Updated by Marcin Kuzminski [CTO] almost 8 years ago
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.
Updated by Marcin Kuzminski [CTO] almost 8 years ago
- Target version changed from v4.5 to v4.6
Updated by Marcin Kuzminski [CTO] over 7 years ago
- Target version changed from v4.6 to v4.7
Updated by Marcin Kuzminski [CTO] over 7 years ago
- Subject changed from Observer role for PR reviewer to Different roles for PR reviewers
Updated by Marcin Kuzminski [CTO] over 7 years ago
- Target version changed from v4.7 to v4.8
Updated by Marcin Kuzminski [CTO] over 7 years ago
- Target version changed from v4.8 to v4.10
Updated by Marcin Kuzminski [CTO] about 7 years ago
- Target version changed from v4.10 to v4.11
Updated by Simon Sarginson` over 4 years ago
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.¶
Updated by Simon Sarginson` over 4 years ago
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.
Updated by Marcin Kuzminski [CTO] over 4 years ago
- Target version changed from v4.11 to v4.20
We're workning on this code now for reviewers, and notifications management, would be good to make those roles too
Updated by Redmine Integration almost 4 years ago
- Status changed from New to Resolved
Commit bfede169912d
by Marcin Kuzminski marcin@rhodecode.com on stable
branch changed this issue.
https://code.rhodecode.com/rhodecode-enterprise-ce/changeset/bfede169912de53b5d6b37c93436d8ac6708b19c