Added pagination support for PullRequest

[lyrixx]

Made a better API than #44 and add tests

[lyrixx]

I implemented a small method to get all PR. Do you want I add this method to this PR:

private function getAllPrs($organisazion, $repositoryName, $status = null)
    {
        $prApi = $this->client->api('pull_request');
        $page = 1;
        $prs = array();
        do {
            $prs = array_merge($prs, $prApi->all($organisazion, $repositoryName, $status, $page, 100));
            $pagination = $this->client->getHttpClient()->getLastResponse()->getPagination();
            $next = isset($pagination['next']) ? $pagination['next'] : false;
            if (!$next) {
                break;
            }
            $url = parse_url($next);
            parse_str($url['query'], $qsa);
            $page = $qsa['page'];
        } while (true);

        return $prs;
    }

Of cource, I will adapt the code ;)

[justinburger]

Seems like the implementation can be done more globally, since most github API's support pagination now. For most people who don't want to deal with pagination, it would be nice to have a global option to up the per-page param. I've got a github enterprise server, so rate limits are not an issue. I'm betting most won't want to deal with pagination regardless.

[dbu]

this is similar but different (and better) then #49 about the repositories call - we should find a general solution to this. i commented in #55 and hope to get a discussion rolling what the correct solution would be.

[lyrixx]

Hello. I do not have to of time for this contribution. Sorry.
What is the state of all theses PR ? Should I close mine ?

[lyrixx]

And BTW, who is the current maintainer of this repository ?

[dbu]

as it looks, whoever is the maintainer is not exactly active on this. i tried to get the attention of @KnpLabs on twitter and in github but so far no luck...

[dbu]

but i propose you leave this PR open until the maintainer shows up and
decides which aproach he likes.

[stloyd]

@pilot @cursedcoder Can you have a look at those? Thanks =)

[pilot]

@stloyd does this applicable?

[dbu]

@pilot did you see the discussion in #55? with this solution, how will a client do pagination? count the results and guess that if he got the maximum there could be a next page and try that?

[pilot]

@dbu I saw and yep it's need to discus, let's leave this PR and move discussion to the #55