require 'oj' is not necessary in base-event-manager.rb

dariocravero / padrino-websockets

Agnostic websockets support for Padrino

MIT License

14 stars 9 forks source link

require 'oj' is not necessary in base-event-manager.rb #3

Open tune opened 10 years ago

tune commented 10 years ago

It seems 'oj' is not mandatory for this gem.

dariocravero commented 10 years ago

Hey @tune, thanks for your message. However, it's being used in on_message and then inside each of the application server implementations to write to the WS. :)

tune commented 10 years ago

If so, I think it's better to add dependency information to 'padrino-websockets.gemspec'.

dariocravero commented 10 years ago

:+1: sorry I totally missed that! :/