+
Skip to content

Add getStatuses repo endpoint #256

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

Closed
wants to merge 1 commit into from
Closed

Add getStatuses repo endpoint #256

wants to merge 1 commit into from

Conversation

pik
Copy link
Contributor

@pik pik commented Nov 13, 2015

Adds the Repo endpoint for https://api.github.com/repos/<user>/<repo>/statuses/<sha>

edit: Build is failing but this seems unrelated -

npm ERR! phantomjs2@2.0.0 install: `node install.js`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the phantomjs2@2.0.0 install script 'node install.js'.```

@AurelioDeRosa
Copy link
Member

Hi @pik and thank you for your PR. Can you please add a test for your method and also update the README to document it?

 * added test for getStatuses
 * updated Readme to include repo.getStatuses method
@pik
Copy link
Contributor Author

pik commented Nov 28, 2015

Updated tests and readme - also I switched to getStatuses from listStatuses since that semed more consistent (?) with the endpoint taking a single sha for an argument.

@AurelioDeRosa AurelioDeRosa changed the title Add listStatuses repo endpoint Add getStatuses repo endpoint Nov 28, 2015
@AurelioDeRosa
Copy link
Member

Merged with a minor update. Thank you for contributing.

@AurelioDeRosa AurelioDeRosa added this to the 0.11.0 milestone Jan 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载