RFR: 218: Enforce check on approval of CSRs
Erik Helin
ehelin at openjdk.java.net
Wed Jan 29 08:08:16 UTC 2020
On Tue, 28 Jan 2020 13:42:42 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:
>> Hi all,
>>
>> please review this patch that adds a CSR bot. The CSR bot allows a reviewer to
>> issue the command `/csr` in a pull request comment. Once issued, the PR cannot
>> be integrated until the CSR for the issue associated with the PR is in state
>> "Closed" and "Approved". A reviewer can cancel this requirement by issuing the
>> command `/csr unneeded` in a pull request comment.
>>
>> Testing:
>> - Added a bunch of new unit tests for the `/csr` command and the CSR bot
>> - `sh gradlew test` passes on Linux x64
>>
>> Thanks,
>> Erik
>
> bots/pr/src/main/java/org/openjdk/skara/bots/pr/CommandWorkItem.java line 98:
>
>> 97: private void processCommand(PullRequest pr, CensusInstance censusInstance, Path scratchPath, String command, Comment comment, List<Comment> allComments) {
>> 98: System.out.println("processing command " + command);
>> 99: var writer = new StringWriter();
>
> Revert?
Right, thanks!
-------------
PR: https://git.openjdk.java.net/skara/pull/385
More information about the skara-dev
mailing list