RFR: 1071: auto-set `/csr` flag from JBS state

Erik Joelsson erikj at openjdk.java.net
Wed Nov 24 19:34:29 UTC 2021


On Wed, 24 Nov 2021 18:05:48 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> Yes there is a closed "Withdrawn" state, but I would probably prefer a Reviewer to issue the `/csr unneeded` in that case. Otherwise, A PR author with the role of Committer could withdraw a CSR that a Reviewer had requested and integrate it without satisfying that requirement.

Yes, that sounds good. What I'm mostly after is what conditions need to be met for the `/csr unneeded` command to succeed. Given this, there are two situations where it should succeed:

1. No CSR issue exists.
2. A CSR issue exists but has been withdrawn.

If neither of those two are true, then issuing `/csr unneeded` should result in an error explaining that a CSR has already been filed and will need to be withdrawn first.

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

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


More information about the skara-dev mailing list