RFR: 443: /reviewer pull request command should accept multiple reviewers

[Erik Helin]

On Thu, 2 Jul 2020 13:14:04 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:

>> Hi all,
>> 
>> please review this patch that makes the `/reviewer` pull request command accept multiple reviewers at once.
>> 
>> Testing:
>> - [x] Added a unit test
>> - [x] `make test` passes on Linux x64
>> 
>> Thanks,
>> Erik
>
> bots/pr/src/main/java/org/openjdk/skara/bots/pr/ReviewerCommand.java line 87:
> 
>> 86:         var reviewers = new ArrayList<Contributor>();
>> 87:         for (var entry : matcher.group(2).split(" ")) {
>> 88:             var reviewer = parseUser(entry, pr, censusInstance);
> 
> Should we allow "," as a separator as well, like the `/issue` and `/cc` commands?

Yeah that makes sense, let me fix ��

-------------

PR: https://git.openjdk.java.net/skara/pull/687