RFR: 574: git-info should use info from commit notifications [v3]
Erik Helin
ehelin at openjdk.java.net
Tue Feb 16 13:29:22 UTC 2021
On Mon, 15 Feb 2021 12:58:47 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> git-info should use info from commit notifications to get the link to the review, not JBS (since not all PR have issues)
>>
>> This required refactoring to move the hosted repo logic from cli/pr/Utils.java to where it can be accessed by all CLI tools. After some deliberating, I left a bunch of trivial wrappers in cli/pr/Utils.java -- otherwise this change would have touched *all* the files in cli/pr.
>>
>> Testing: ad hoc testing of "git info" on some arbitrary Skara commits.
>
> Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last revision:
>
> Add test for reviewUrl parsing
Looks good modulo some whitespace issues found by jcheck :robot:
-------------
Marked as reviewed by ehelin (Reviewer).
PR: https://git.openjdk.java.net/skara/pull/1011
More information about the skara-dev
mailing list