RFR: 234: Allow more granular jcheck reviewer configuration
Erik Helin
ehelin at openjdk.java.net
Mon Jan 20 14:51:50 UTC 2020
Hi all,
please review this patch that allows for more granular reviewers configuration
with jcheck. With this patch a project can now specify exactly how many
reviewers are need for each role, for example:
[checks "reviewer"]
reviewer = 1
committer = 1
author = 1
The above configuration would require three reviewers in total, out of which one
must be reviewer, one must be committer and one must be author. This granularity
of configuration is needed to fully implement [PR 364](https://git.openjdk.java.net/skara/pulls/364).
I made sure that the patch is backwards compatible, we still support the old
configuration syntax:
[checks "reviewer"]
minimum = 1
role = reviewer
The above will simply be translated to the following configuration:
[checks "reviewer"]
reviewer = 1
Thanks,
Erik
## Testing
- [x] Added a number of additional unit tests
-------------
Commits:
- 7ce157a2: skara-234
Changes: https://git.openjdk.java.net/skara/pull/368/files
Webrev: https://webrevs.openjdk.java.net/skara/368/webrev.00
Issue: https://bugs.openjdk.java.net/browse/SKARA-234
Stats: 260 lines in 6 files changed: 216 ins; 11 del; 33 mod
Patch: https://git.openjdk.java.net/skara/pull/368.diff
Fetch: git fetch https://git.openjdk.java.net/skara pull/368/head:pull/368
PR: https://git.openjdk.java.net/skara/pull/368
More information about the skara-dev
mailing list