RFR: pr: add /backport commit command
Robin Westberg
rwestberg at openjdk.java.net
Thu Nov 5 14:30:52 UTC 2020
On Thu, 5 Nov 2020 12:58:24 GMT, Erik Helin <ehelin at openjdk.org> wrote:
> Hi all,
>
> please review this patch that adds the `/backport` [commit command](https://wiki.openjdk.java.net/display/SKARA/Commit+Commands#CommitCommands-/backport).
>
> Testing:
> - [x] Added four new unit tests
> - [x] `make test` passes on Linux x64
>
> Thanks,
> Erik
Looks good, only some minor comments!
bots/pr/src/main/java/org/openjdk/skara/bots/pr/BackportCommand.java line 64:
> 62: var username = command.user().username();
> 63: if (censusInstance.contributor(command.user()).isEmpty()) {
> 64: reply.println("@" + username + " only OpenJDK [contributor](https://openjdk.java.net/bylaws#contributor) can use the `/backport` command");
Perhaps "only an" or only .. contributors?
bots/pr/src/main/java/org/openjdk/skara/bots/pr/BackportCommand.java line 64:
> 62: var username = command.user().username();
> 63: if (censusInstance.contributor(command.user()).isEmpty()) {
> 64: reply.println("@" + username + " only OpenJDK [contributor](https://openjdk.java.net/bylaws#contributor) can use the `/backport` command");
The command handler will prefix the reply with `@command-issuer` so should not need to do that manually.
-------------
Marked as reviewed by rwestberg (Reviewer).
PR: https://git.openjdk.java.net/skara/pull/945
More information about the skara-dev
mailing list