[Approved] RFR: 80: Allow valid bug ID format to be configurable

Robin Westberg rwestberg at openjdk.org
Thu Sep 26 11:54:07 UTC 2019


On Wed, 25 Sep 2019 13:50:50 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> JBS issue: https://bugs.openjdk.java.net/browse/SKARA-80
> 
> This provides a configuration option to define the pattern for a valid issue ID as follows. This starts with the idea @edvbld mentioned in PR #99, although I felt it was cleaner to leave the default issue format as is, and use the more restrictive pattern for the "legacy" jdk conf file.
> 
> The syntax of the new pattern is:
> 
> [checks "issues"]
> pattern=<PATTERN_STRING>
> 
> The default pattern is:
> 
> [checks "issues"]
> pattern=^(([A-Z][A-Z0-9]+-)?[0-9]+): (\\S.*)$
> 
> For legacy JDK `.jcheck/conf` files, the pattern is as follows to match the issue ID rules in `jcheck.py`:
> 
> [checks "issues"]
> pattern=^([124-8][0-9]{6}): (\\S.*)$
> 
> ----------------
> 
> Commits:
>  - a12ec8ee: 80: Allow valid bug ID format to be configurable
> 
> Changes: https://git.openjdk.java.net/skara/pull/163/files
>  Webrev: https://webrevs.openjdk.java.net/skara/163/webrev.00
>   Issue: https://bugs.openjdk.java.net/browse/SKARA-80
>   Stats: 321 lines in 5 files changed: 315 ins; 1 del; 5 mod
>   Patch: https://git.openjdk.java.net/skara/pull/163.diff
>   Fetch: git fetch https://git.openjdk.java.net/skara pull/163/head:pull/163

Looks good, thanks for fixing this! Noticed a pre-existing typo, could perhaps fix that while we're at it..

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

Approved by rwestberg (Reviewer).

PR: https://git.openjdk.java.net/skara/pull/163


More information about the skara-dev mailing list