RFR: 1690: Make jcheck cli able to use conf from workspace or other commit [v3]

Zhao Song zsong at openjdk.org
Thu Dec 1 16:27:50 UTC 2022


On Thu, 1 Dec 2022 14:08:52 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> vcs/src/main/java/org/openjdk/skara/vcs/git/GitRepository.java line 1664:
>> 
>>> 1662: 
>>> 1663:     @Override
>>> 1664:     public Optional<Hash> wholeHash(String rev) {
>> 
>> Isn't this just `::resolve(String)`?
>
> What I mean is, you could use resolve instead of introducing this new method.

Oh, I don't know why I didn't see some of your review yesterday. Ah, right, I will use resolve and delete the new method

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

PR: https://git.openjdk.org/skara/pull/1428


More information about the skara-dev mailing list