Navigation Menu

Skip to content
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

contributor api binding #114

Merged
merged 1 commit into from May 23, 2016
Merged

contributor api binding #114

merged 1 commit into from May 23, 2016

Conversation

sevenEng
Copy link
Contributor

@sevenEng sevenEng commented May 6, 2016

try to solving issue #112

according to the github doc https://developer.github.com/v3/repos/statistics/#get-contributors-list-with-additions-deletions-and-commit-counts, request would be responded with 202 (and empty body) when the contributor data need to be calculated and haven't been cached, under this situation, return an empty list

omitted some fields in the response, suggestions welcome : )

@samoht
Copy link
Member

samoht commented May 21, 2016

I've restarted the failed TravisCI build. Not sure why camomille failed on Windows..

@dsheets
Copy link
Member

dsheets commented May 23, 2016

Camomile is still failing on Windows. I will merge this anyway.

Thanks, Qi Li, this is a nice contribution and really helps. I have a few nitpicks but I will address them in a subsequent PR.

@dsheets dsheets merged commit 76987be into mirage:master May 23, 2016
This was referenced May 24, 2016
@sevenEng
Copy link
Contributor Author

Glad I can help : ) Thanks for merging 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants