Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Net-GitHub CPAN distribution.

Report information
The Basics
Id: 99365
Status: open
Priority: 0/
Queue: Net-GitHub
People
Owner: Nobody in particular
Requestors: mschwern [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.69
Fixed in: (no value)

History Show all quoted text
  Tue Oct 07 19:07:48 2014 mschwern [...] cpan.org - Ticket created
Subject: Increase the items per page
Gitpan sometimes has to list a lot of repositories, like 20,000+.  At the default 30 items per query this is nearly 1000 queries.  Github supports up to 100 per page.  https://developer.github.com/v3/#pagination

It would be very useful if Net::GitHub would allow setting 'per_page'.  I would even suggest defaulting it to 100 as in most cases one wants to do the least queries.

I'd do it myself, but I'm not sure where it fits in the API.
  Tue Oct 07 20:11:03 2014 fayland [...] cpan.org - Correspondence added
Hi do you think something like this will work? https://github.com/fayland/perl-net-github/commit/13460bf2d461b728b5897ebb7b19602b7159cba3 If yes, I'll be glad to release a new version. feedback welcome. Thanks
  Tue Oct 07 20:11:03 2014 The RT System itself - Status changed from 'new' to 'open'
  Wed Oct 08 13:02:23 2014 schwern [...] pobox.com - Correspondence added
Subject: Re: [rt.cpan.org #99365] Increase the items per page
Date: Wed, 08 Oct 2014 10:02:00 -0700
To: bug-Net-GitHub [...] rt.cpan.org
From: Michael G Schwern <schwern [...] pobox.com>
My initial reaction is to be wary of adding it onto every GET request. But on thinking about it, it's difficult to know what call may paginate. And the API should ignore it if it doesn't apply.
  Wed Oct 08 19:54:37 2014 fayland [...] cpan.org - Correspondence added
On Wed Oct 08 13:02:23 2014, schwern@pobox.com wrote: Show quoted text
> My initial reaction is to be wary of adding it onto every GET request. > But on thinking about it, it's difficult to know what call may paginate. > And the API should ignore it if it doesn't apply.
I'll check other language implementation and Pithub this weekend to see how do they work. will update you later. Thanks