RFR: 1452: Can never notify on first commit [v3]
Zhao Song
zsong at openjdk.org
Thu Oct 6 18:44:17 UTC 2022
On Thu, 6 Oct 2022 17:48:30 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
>> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>>
>> set a threshold to determine whether should notify on first commit
>
> vcs/src/main/java/org/openjdk/skara/vcs/hg/HgRepository.java line 1500:
>
>> 1498: @Override
>> 1499: public int getExistingCommits() throws IOException {
>> 1500: //TODO:: implement it later, return 10 here so that it will maintain the previous behavior
>
> Should probably throw `UnsupportedOperationException` if not implementing this. I know we aren't using this for any HG repos in production. Is there any test that would trip this?
>
> You could also implement it. A reasonable command line would be
>
> hg id --num --rev tip
>
> and then add 1 to that number, because the first commit in HG is numbered 0.
I don't think we would create any HG repo in tests, so no test will trip this.
I am not familiar with HG repo, so I didn't implement it.
I will use the command you mentioned to implement it. Thanks!
-------------
PR: https://git.openjdk.org/skara/pull/1385
More information about the skara-dev
mailing list