Skip to content

Use new /release/top_uploaders API endpoint #1909

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 8, 2017
Merged

Use new /release/top_uploaders API endpoint #1909

merged 1 commit into from
Jun 8, 2017

Conversation

@mickeyn
Copy link
Contributor

@mickeyn mickeyn commented Jun 6, 2017

Replace sending query with use of the new API endpoint.
@mickeyn
Use new /release/top_uploaders API endpoint
ba58179 
Replace sending query with use of the new API endpoint.
@mickeyn mickeyn requested a review from oalders June 6, 2017 21:08
@coveralls
Copy link

coveralls commented Jun 7, 2017

Coverage Status

Coverage remained the same at 52.893% when pulling ba58179 on mickey/top_uploaders_endpoint into 4f92929 on master.
oalders
oalders approved these changes Jun 8, 2017
View reviewed changes
Copy link
Member

@oalders oalders left a comment

Choose a reason for hiding this comment

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

👍
@oalders oalders merged commit 169b80a into master Jun 8, 2017
@oalders oalders deleted the mickey/top_uploaders_endpoint branch June 8, 2017 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants

@mickeyn @coveralls @oalders