RFR: 1406: Two GitHub APIs don't return `patch` field when `per_page` argument exceeds 70 [v2]

Guoxiong Li gli at openjdk.java.net
Fri Apr 22 15:31:38 UTC 2022


On Fri, 22 Apr 2022 14:00:35 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add comment.
>
> forge/src/main/java/org/openjdk/skara/forge/github/GitHubPullRequest.java line 740:
> 
>> 738:     public Diff diff() {
>> 739:         var files = request.get("pulls/" + json.get("number").toString() + "/files")
>> 740:                            .param("per_page", "50")
> 
> Can we add a comment explaining the need for per_page? Something like:
> 
> "Need to specify an explicit per_page < 70 to guarantee that we get patch information in the result set."

Added.

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

PR: https://git.openjdk.java.net/skara/pull/1302


More information about the skara-dev mailing list