RFR: 1690: Make jcheck cli able to use conf from workspace or other commit [v3]
Zhao Song
zsong at openjdk.org
Thu Dec 1 00:20:56 UTC 2022
On Wed, 30 Nov 2022 23:29:41 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
>> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>>
>> implemented case 5
>
> vcs/src/main/java/org/openjdk/skara/vcs/git/GitRepository.java line 1687:
>
>> 1685: @Override
>> 1686: public Commit staged() throws IOException {
>> 1687: var author = new Author("jcheck", "jcheck at none.none");
>
> For the default user and email, there is already a `::userName` method. You can create one for email as well, and then use these values as fallbacks (`Optional.orElse`).
> There are a lot of minor things, but all in all this turned out better than I had imagined. Looking forward to using it.
>
> Have you tried all the options against a repository with various degrees of local changes in it?
Yes, I tried all the options and all the combination of options.
-------------
PR: https://git.openjdk.org/skara/pull/1428
More information about the skara-dev
mailing list