RFR: 1984: SKARA bot couldn't handle jep command if the jep doesn't have JEP Number
Kevin Rushforth
kcr at openjdk.org
Wed Aug 2 22:01:07 UTC 2023
On Wed, 2 Aug 2023 21:27:13 GMT, Zhao Song <zsong at openjdk.org> wrote:
> A user issued command /jep JDK-8310626 in https://github.com/openjdk/jdk/pull/15103 and skara bot kept throwing exceptions because the JEP doesn't have a JEP number.
>
> @kevinrushforth suggested that it makes more sense to make skara bot be able to associate a PR with a JEP before it is a Candidate.
Looks good. I left one minor comment about the message string, but I approved it anyway.
bots/pr/src/main/java/org/openjdk/skara/bots/pr/JEPCommand.java line 147:
> 145: // The current issue status may be "Draft", "Submitted", "Candidate", "Proposed to Target", "Proposed to Drop" or "Closed without Delivered"
> 146: if (jepNumber.equals("NotAllocated")) {
> 147: reply.println("This pull request will not be integrated until the [" + jbsIssue.id()
Minor: I would add the word "JEP" before the `jbsIssue.id()` here.
-------------
Marked as reviewed by kcr (Reviewer).
PR Review: https://git.openjdk.org/skara/pull/1541#pullrequestreview-1559898509
PR Review Comment: https://git.openjdk.org/skara/pull/1541#discussion_r1282456083
More information about the skara-dev
mailing list