Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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! [email protected] install: `node install.js`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the [email protected] 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