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

Add superfast responder #52

Closed
wants to merge 1 commit into from
Closed

Add superfast responder #52

wants to merge 1 commit into from

Conversation

sjayanna
Copy link

@sjayanna sjayanna commented Sep 2, 2016

@scrogson This responder is very useful!

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.006% when pulling 9227d4e on sjayanna:add-superfast-responder into 230a4c2 on hedwig-im:master.

@scrogson
Copy link
Member

scrogson commented Sep 7, 2016

Hi @sjayanna,

Thanks for putting this PR together. At this time, I'd like to encourage you to make this into a separate library on hex. I want to limit the number of included responders in hedwig itself. I will probably end up removing all except for the help and ping responders.

@scrogson scrogson closed this Sep 7, 2016
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.

3 participants