RFR: Add --no-token flag to git-pr

Erik Helin ehelin at openjdk.java.net
Tue Nov 26 11:08:05 UTC 2019

On Tue, 26 Nov 2019 11:03:43 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hi all,
> please review this pull request that adds a `--no-token` flag to `git-pr`. This is useful for those merely wanting to try out the read-only operations of `git-pr` (such as `git pr list` or `git pr show`) or for those that do not want to create a personal access token (PAT). The downside for _not_ using a token for read-only operations is that GitHub quite aggressively rate limits the number of REST API HTTP operations (seems like 60 REST API HTTP operations per minute at the moment).
> Thanks,
> Erik
> ## Testing
> - Manual testing of various `git-pr` subcommands
> ----------------
> Commits:
>  - ac5efe8e: Add --no-token flag
>  - dab55bda: Support read-only commands in git-pr
> Changes: https://git.openjdk.java.net/skara/pull/270/files
>  Webrev: https://webrevs.openjdk.java.net/skara/270/webrev.00
>   Stats: 115 lines in 4 files changed: 43 ins; 41 del; 31 mod
>   Patch: https://git.openjdk.java.net/skara/pull/270.diff
>   Fetch: git fetch https://git.openjdk.java.net/skara pull/270/head:pull/270

Robin please note that this patch also makes some changes to `GitHubHost.java`, `GitHubRepository.java` and how those classes handle the installation token (I also cleaned up an old test that didn't require a token).

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

More information about the skara-dev mailing list