Skip to content
This repository has been archived by the owner on Apr 30, 2019. It is now read-only.

Typo fix in README #229

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

rhysd
Copy link

@rhysd rhysd commented Oct 9, 2015

No description provided.

@@ -475,7 +475,7 @@ Useful behind (corporate) proxies that act like man-in-the middle on https conne

- "**token**" - Github OAuth token:

The OAuth token can be used to boost the Github API rate-limit from 60 to 5000 (non-cached) requests per hour. The is token needs just ['read-only access to public information'](http://developer.github.com/v3/oauth/#scopes) so no additional OAuth scopes are necessary.
The OAuth token can be used to boost the Github API rate-limit from 60 to 5000 (non-cached) requests per hour. The token needs just ['read-only access to public information'](http://developer.github.com/v3/oauth/#scopes) so no additional OAuth scopes are necessary.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The phrase "The token needs just" doesn't quite make sense still. Did you want to change it to read "The token only needs 'read-only access to public information', no additional OAuth scopes are necessary."?

@graingert graingert mentioned this pull request Mar 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants