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

[Github] Populate Service Test #976

Closed
wants to merge 16 commits into from

Conversation

webcaetano
Copy link
Contributor

@webcaetano webcaetano commented Apr 29, 2017

Multiple github service tests

Merge this PR will also auto merge #745 and #756

Related to #937

@webcaetano
Copy link
Contributor Author

I'll remake this soon in order to eliminate unnecessary conflicts

@webcaetano
Copy link
Contributor Author

Remaked

@paulmelnikow
Copy link
Member

paulmelnikow commented May 6, 2017

Hey, just wanted to say that I started reviewing this. Will hopefully finish writing comments and post them this weekend.

@webcaetano
Copy link
Contributor Author

ok, thanks

paulmelnikow added a commit that referenced this pull request Sep 22, 2017
Work by webcaetano (cherry-picked by paulmelnikow)
@paulmelnikow
Copy link
Member

Cherry-picked and merged via 8ec2b6b.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
service-badge Accepted and actionable changes, features, and bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants