RFR: 2278: Issuestitle Check shouldn't treat special character as leading lowercase letter [v2]

Kevin Rushforth kcr at openjdk.org
Tue Jun 4 20:20:56 UTC 2024


On Tue, 4 Jun 2024 20:16:19 GMT, Zhao Song <zsong at openjdk.org> wrote:

>> Is the `:?` needed? It seems redundant, but I'm not a regex expert.
>
> I think Erik has a typo here, should be `"[a-z]+(?:\\h.*)?"`?

On closer look, the "`:?`" won't do what we want. It will match one of the patterns we explicitly do not want to match: `"abc: def"`. So I think that should just be: `"[a-z]+(\\h.*)?"`

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

PR Review Comment: https://git.openjdk.org/skara/pull/1656#discussion_r1626557165


More information about the skara-dev mailing list