RFR: 383: More fuzzy matching of bug ids in PR titles
Robin Westberg
rwestberg at openjdk.java.net
Thu May 14 14:34:41 UTC 2020
On Thu, 14 May 2020 13:56:00 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> Hi all,
>>
>> Please review this change that allows a more relaxed matching of bug ids in PR titles. Also contains a bit of
>> refactoring to make it explicit when the project name could be part of the bug id. There were several places that
>> expected the project name to not be present (as they would fetch the issue using the key from a specific project) -
>> these now explicitly use the shortId. Best regards, Robin
>
> vcs/src/main/java/org/openjdk/skara/vcs/openjdk/Issue.java line 89:
>
>> 88: public String toShortString() {
>> 89: return id + ": " + description;
>> 90: }
>
> Bit confusing here when also looking at the toString method impl. Maybe you should rename the `id` field to `shortId`
> now? (to match the method names)
Makes sense, will change it.. There's probably a better name than `shortId` waiting to be discovered out there, but I
didn't find it..
-------------
PR: https://git.openjdk.java.net/skara/pull/623
More information about the skara-dev
mailing list