New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add --no-token flag to git-pr #270
Conversation
👋 Welcome back ehelin! A progress list of the required criteria for merging this PR into |
Robin please note that this patch also makes some changes to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Perhaps a typo, but the anonymous rate limit is actually 60 requests per hour (not minute) so even worse. :)
@edvbld This change can now be integrated. The commit message will be:
|
/integrate |
Mailing list message from Erik Helin on skara-dev: Changeset: 5c3f278 Add --no-token flag to git-pr Reviewed-by: rwestberg ! cli/src/main/java/org/openjdk/skara/cli/GitPr.java |
Hi all,
please review this pull request that adds a
--no-token
flag togit-pr
. This is useful for those merely wanting to try out the read-only operations ofgit-pr
(such asgit pr list
orgit 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
git-pr
subcommandsProgress
Approvers