RFR: 1554: Fold JEPBot into PR bot [v2]
Erik Joelsson
erikj at openjdk.org
Tue Jun 27 13:08:10 UTC 2023
On Mon, 26 Jun 2023 22:25:46 GMT, Zhao Song <zsong at openjdk.org> wrote:
>> Erik Joelsson has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove jep module
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/skara/pull/1535#discussion_r1243698575
More information about the skara-dev
mailing list