kostya / eye

Process monitoring tool. Inspired from Bluepill and God.
MIT License
1.19k stars 86 forks source link

slack notifier doesn't find 'slack-notifier' #139

Closed blackjid closed 9 years ago

blackjid commented 9 years ago

I'm getting an error when trying to use the slack notification.

cannot load such file -- slack-notifier

I think the gem slack-notifier needs to be a runtime dependency rather a development dependency. https://github.com/kostya/eye/blob/46dc7a7164b9b51003dbfacd0b77a544e24d28e3/eye.gemspec#L38

kostya commented 9 years ago

i think it should not depend on it. you should install 'slack-notifier' by yourself. i think better to create new gem that depend on eye and slack-notifier, like this: https://github.com/tmeinlschmidt/eye-hipchat, it easier to maintain, and remove slack notifier from eye.

blackjid commented 9 years ago

I totally agree.

tmeinlschmidt commented 9 years ago

hey all, I already added support for slack - https://github.com/tmeinlschmidt/eye-slack :)