RFR: forge: support commit comments
Erik Helin
ehelin at openjdk.java.net
Fri Jun 5 11:50:58 UTC 2020
On Fri, 5 Jun 2020 08:30:08 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:
>> Hi all,
>>
>> please review this patch that adds the possibility to add a comment on a particular commit to `HostedRepository` via
>> the method `addCommitComment`. To start out with I've only added support for adding a general comment about a whole
>> commit, not a comment for a particular line in a commit. The comments for a commit can also be listed with the method
>> `commitComments`.
>> Testing:
>> - Not that much, later patches will build upon the APIs introduced in this patch
>>
>> Thanks,
>> Erik
>
> forge/src/main/java/org/openjdk/skara/forge/CommitComment.java line 2:
>
>> 1: /*
>> 2: * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
>> 3: * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>
> 2020!
Thanks, will fix!
> forge/src/main/java/org/openjdk/skara/forge/CommitComment.java line 65:
>
>> 64: */
>> 65: public Optional<Integer> position() {
>> 66: return position == -1 ? Optional.empty() : Optional.of(position);
>
> Perhaps drop this one? The other API's only expose a line number
Ok, yeah, sounds good
-------------
PR: https://git.openjdk.java.net/skara/pull/639
More information about the skara-dev
mailing list