search

hedwig-im / hedwig

An Adapter-based Bot Framework for Elixir Applications
MIT License
650 stars 73 forks source link

Add superfast responder #52

Closed sjayanna closed 7 years ago

sjayanna commented 7 years ago

@scrogson This responder is very useful!

coveralls commented 7 years ago

Coverage Status

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

scrogson commented 7 years ago

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.