RFR: 2089: Warn when bug fixVersion doesn't match .jcheck/conf version in PR [v2]
Zhao Song
zsong at openjdk.org
Thu Nov 9 20:31:17 UTC 2023
On Thu, 9 Nov 2023 19:37:35 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
>> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>>
>> add fixVersionOverride option
>
> bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestBotFactory.java line 247:
>
>> 245: }
>> 246:
>> 247: if (repo.value().contains("fixVersionOverride")) {
>
> I think the config option should reflect that we are turning off this warning specifically, rather than that we have configured a different fixVersion in the notifier. There are more situations where this warning doesn't apply (see bug comments).
>
> How about `versionMismatchWarning` and I think we should even default it to `false` and selectively add it to the repos we want it on.
Sure, thanks
-------------
PR Review Comment: https://git.openjdk.org/skara/pull/1585#discussion_r1388549138
More information about the skara-dev
mailing list