-
Enhancement
-
Resolution: Unresolved
-
P3
-
None
-
None
-
None
I have noted that it has become more common to signal "I approve of parts of this PR, but you need someone else too to look at it" by raising the number of reviewers required. I think that is a good pattern. Just writing a comment "you need more reviewers" can be missed, especially by fresh committers in the project.
But. Right now this requires you to first look up how many reviewers there are, and then create a comment with a number higher than that. I think Skara should be able to help out here. I suggest adding one (or more, as aliases) of the following subcommands to /reviewers:
/reviewers inc
/reviewers +
/reviewers increment
which would then take the current number of reviewers, and set the new count to be one higher. Especially coupled withSKARA-1789, this would make for an easy way to review a PR but not give a full go-ahead for integration.
But. Right now this requires you to first look up how many reviewers there are, and then create a comment with a number higher than that. I think Skara should be able to help out here. I suggest adding one (or more, as aliases) of the following subcommands to /reviewers:
/reviewers inc
/reviewers +
/reviewers increment
which would then take the current number of reviewers, and set the new count to be one higher. Especially coupled with