RFR: 1554: Fold JEPBot into PR bot [v2]

Zhao Song zsong at openjdk.org
Tue Jun 27 16:08:11 UTC 2023


On Tue, 27 Jun 2023 13:01:44 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java line 330:
>> 
>>> 328: 
>>> 329:         var issueProject = issueProject();
>>> 330:         if (issueProject != null) {
>> 
>> Do we still need to check `issueProject ` here?
>
> I'm not completely sure, but if we don't, there is a risk that we change behavior when a bot is configured without an issue tracker/project. It would be a good followup to really investigate if that should be a valid configuration and how that is supposed to work.

Sure. It's no harm to keep the check here.

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

PR Review Comment: https://git.openjdk.org/skara/pull/1535#discussion_r1244001411


More information about the skara-dev mailing list