RFR: 7903659 Verify Problemlist contents

Ludvig Janiuk lujaniuk at openjdk.org
Fri Mar 8 16:15:14 UTC 2024


On Fri, 8 Mar 2024 15:31:14 GMT, Christian Stein <cstein at openjdk.org> wrote:

>> @sormuras So, having looked at -check: it "DOES NOT" run the tests. But I intend problemlist verification to be run as part of testing, e.g. in CI's. So I think this warrants existing outside of -check.
>> 
>> I agree -check should perhaps be extented to cover problemlists too, but that is not the scope of this PR.
>> 
>> Including -match seems like a very reasonable thing to add to this PR.
>
>> I agree `-check` should perhaps be extented to cover `problemlists` too, but that is not the scope of this PR.
> 
> Okay. Let's introduce new options, including the "--long-word" variant: `--verify-exclude` in this PR.

Ok so just after posting my comment about `-verifyexclude` vs `--verify-exclude`, I saw @sormuras comment 

> Let's introduce new options, including the "--long-word" variant: --verify-exclude in this PR.

I'm fine with this, but it's not what one infers from the current set of flags. So could we then document the intended style of new flags somewhere?

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

PR Comment: https://git.openjdk.org/jtreg/pull/181#issuecomment-1985972050


More information about the jtreg-dev mailing list