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