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

Zhao Song zsong at openjdk.org
Tue Jun 4 20:14:58 UTC 2024


On Tue, 4 Jun 2024 19:38:47 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   update
>
> jcheck/src/main/java/org/openjdk/skara/jcheck/IssuesTitleCheck.java line 91:
> 
>> 89: 
>> 90:     private boolean hasLeadingLowerCaseLetter(String description) {
>> 91:         return ALL_LOWER_CASE_PATTERN.matcher(description.split(" ")[0]).matches();
> 
> I presume that `description` has been `trim`med? If so, then this should be fine without additional checks.

Yes, the description has been trimmed

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

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


More information about the skara-dev mailing list