Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Nov 1, 2017. It is now read-only.

Add required permissions for repo keys#163

Closed
pythonandchips wants to merge2 commits intogithub:masterfrompythonandchips:master

Conversation

@pythonandchips
Copy link

There is no documentation on what authorisation scope is required when accessing the api. Took me a while to figure it out so though I would add something to clear it up.

Add instruction to add repo:keys to authorization to ensure requests will work
made authorization clearer
@pengwynn
Copy link
Contributor

Thanks for the patch. I believe you needrepo scope sincerepo:keys does not exist. I'd love to fully document the scopes required for each method, but I think we're better off automating that via the build process to to pull from theX-Accepted-OAuth-Scopesheader or something. I'm going to close this until we can do it across the board.

@pythonandchips
Copy link
Author

Indeed that would be a lot better if it was generated.

Interestingly when I tried to add a key with a repo scope it wouldn't work but with repo:keys it did. need to try it again with just the repo scope.

Cheers

Stay Frosty
Colin
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Sunday, 21 October 2012 at 14:26, Wynn Netherland wrote:

Thanks for the patch. I believe you need repo scope since repo:keys does not exist. I'd love to fully document the scopes required for each method, but I think we're better off automating that via the build process to to pull from the X-Accepted-OAuth-Scopes header (http://developer.github.com/v3/oauth/#scopes) or something. I'm going to close this until we can do it across the board.


Reply to this email directly or view it on GitHub (#163 (comment)).

gjtorikian added a commit that referenced this pull requestNov 24, 2014
Update CI guide so that commit statuses are created on the base repo of pull requests
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pythonandchips@pengwynn

[8]ページ先頭

©2009-2025 Movatter.jp