RFR: 1419: The Skara PR bot should not block on a JEP if not enabled for a repo
Erik Joelsson
erikj at openjdk.java.net
Tue May 3 12:54:56 UTC 2022
On Tue, 3 May 2022 00:22:47 GMT, Guoxiong Li <gli at openjdk.org> wrote:
> Hi all,
>
> If a repository doesn't support JEP and someone issues the `/jep` command in the PR of this repo, the bot should replies a comment instead of blocking the PR. This patch adds a jep config to the PR bot and adjusts the test cases.
>
> Thanks for taking the time to review.
>
> Best Regards,
> -- Guoxiong
bots/pr/src/main/java/org/openjdk/skara/bots/pr/JEPCommand.java line 83:
> 81: List<Comment> allComments, PrintWriter reply, List<String> labelsToAdd, List<String> labelsToRemove) {
> 82: if (!bot.enableJep()) {
> 83: reply.println("this repository is not allowed to use the `jep` command.");
Suggestion:
reply.println("This repository has not been configured to use the `jep` command.");
-------------
PR: https://git.openjdk.java.net/skara/pull/1310
More information about the skara-dev
mailing list