How to combine comments for mutiple lines

12 views
Skip to first unread message

xu...@yahoo-inc.com

unread,
Sep 1, 2014, 4:24:49 AM9/1/14
to repo-d...@googlegroups.com

I have a use case as below:

 for line:  150-180,  98-105

they have same comments "xxxxxxx",   Can I combine them together by REST API  by setting CommentInput?


 

David Pursehouse

unread,
Sep 1, 2014, 4:55:08 AM9/1/14
to xu...@yahoo-inc.com, repo-d...@googlegroups.com
Yes, this is possible. The UI uses this, but the documentation did not
include an example of it until recently.

The documentation currently deployed on gerrit-review includes it:

https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#set-review

xu...@yahoo-inc.com

unread,
Sep 1, 2014, 5:00:47 AM9/1/14
to repo-d...@googlegroups.com, xu...@yahoo-inc.com
Sorry I do not think so.

My use case is
150-180 has comment "xxxx"
98-105   has comment "xxxx"

combine them together

your example only can resolve 150-180 has comment with "xxxxx" OR 98-105   has comment "xxxx"

David Pursehouse

unread,
Sep 1, 2014, 5:11:07 AM9/1/14
to xu...@yahoo-inc.com, repo-d...@googlegroups.com
On 09/01/2014 06:00 PM, xu...@yahoo-inc.com wrote:
> Sorry I do not think so.
>
> My use case is
> 150-180 has comment "xxxx"
> 98-105 has comment "xxxx"
>
> combine them together
>
> your example only can resolve 150-180 has comment with "xxxxx" OR
> 98-105 has comment "xxxx"
>

So you mean you want to have one comment, and apply the same comment to
two separate ranges?

No, that is not possible. You would have to specify two comments, with
the same message text, but different range parameters.

Reply all
Reply to author
Forward
0 new messages