Integrated: 1252: Improve the output message about the command `csr`

Guoxiong Li gli at openjdk.java.net
Mon Nov 22 17:17:13 UTC 2021


On Mon, 22 Nov 2021 09:58:13 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> Hi all,
> 
> Currently, when a user, who is not the author of the pr and is not a reviewer, uses the command `/csr unneeded`, the bot will output the message `only the pull request author and [Reviewers](https://openjdk.java.net/bylaws#reviewer) are allowed to require a CSR.`. The message is good when meeting the command `/csr` or `/csr needed` but it is not good when meeting the command `/csr unneeded` or `/csr uneeded`.
> 
> This patch changes the message `are allowed to require a CSR` to `are allowed to use the csr command` and adds the corresponding test cases.
> 
> Thanks for taking the time to review.
> 
> Best Regards,
> -- Guoxiong

This pull request has now been integrated.

Changeset: 9b1f05c0
Author:    Guoxiong Li <gli at openjdk.org>
Committer: Erik Joelsson <erikj at openjdk.org>
URL:       https://git.openjdk.java.net/skara/commit/9b1f05c0e929b543f495889e2e66f0c56857e781
Stats:     28 lines in 2 files changed: 24 ins; 0 del; 4 mod

1252: Improve the output message about the command `csr`

Reviewed-by: erikj

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

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


More information about the skara-dev mailing list