On Wed, 1 Nov 2023 15:16:54 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
> A recent change to Skara in processing of the .jcheck/conf file > ([SKARA-2080](https://bugs.openjdk.org/browse/SKARA-2080)) introduced a bug > in processing the `[checks "reviewers"]` section that causes the `/reviewers` > command to be ignored in some cases. This is tracked by > [SKARA-2088](https://bugs.openjdk.org/browse/SKARA-2088). The reason we > tripped over this bug is that the `.jcheck/conf` file used by jfx (including > various update release repos) uses an unrecognized key, "minimum", instead of > the expected "reviewers" key to indicate the minimum number of reviewers. We > should fix this to be consistent with what Skara expects given the intent of > our `[checks "reviewers"]` section. Looks good! ------------- Marked as reviewed by ehelin (no project role). PR Review: https://git.openjdk.org/jfx/pull/1275#pullrequestreview-1708522921
