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

Removes unnecessary argument to DeregisterByTask #105

Merged
merged 1 commit into from
Aug 23, 2016

Conversation

pbetkier
Copy link
Contributor

Change base to develop after #104 gets merged.

@coveralls
Copy link

coveralls commented Aug 19, 2016

Coverage Status

Coverage remained the same at 84.204% when pulling 3b3ba19 on remove_unnecessary_deregister_argument into 780fb4b on find_service_by_task_using_tags.

@janisz
Copy link
Contributor

janisz commented Aug 19, 2016

LGTM 👍

@dankraw
Copy link
Contributor

dankraw commented Aug 22, 2016

👍

@pbetkier pbetkier changed the base branch from find_service_by_task_using_tags to develop August 23, 2016 08:14
@pbetkier pbetkier force-pushed the remove_unnecessary_deregister_argument branch from 3b3ba19 to ffd5855 Compare August 23, 2016 08:33
@coveralls
Copy link

coveralls commented Aug 23, 2016

Coverage Status

Changes Unknown when pulling ffd5855 on remove_unnecessary_deregister_argument into * on develop*.

@pbetkier pbetkier merged commit 6e0685a into develop Aug 23, 2016
@pbetkier pbetkier deleted the remove_unnecessary_deregister_argument branch August 23, 2016 09:00
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.

5 participants