search

hedwig-im / hedwig_slack

Slack Adapter for Hedwig
MIT License
45 stars 30 forks source link

Fix Elixir warnings #7

Closed lucidstack closed 7 years ago

lucidstack commented 7 years ago

Hi there, thank you for making hedwig(_slack)!

While working with it, I noticed some warnings on Elixir 1.3.2 (mostly unused variables here and there), and decided to clear these up with this PR. The commits are pretty self-explanatory, but please don't hesitate to ask for clarifications!

Thank you again! 👋

scrogson commented 7 years ago

Hi there @lucidstack, thank you for you interest in helping! I'm currently in the middle of overhauling all of this in order to fix #5. Since merging this will just cause conflicts, I'm going to close and ensure that I clean these things up before the next release.