Dependencies about sharejs HOT 5 CLOSED

Good idea. I think package.json will pull in nodeunit automatically, but obviously there's no npm package for the whole redis database.

from sharejs.

Perhaps we should make bin/sharejs use memory db by default?

from sharejs.

Yeah, maybe. Or it should try and load redis, and fall back to memory if thats missing.

from sharejs.

Sounds good :)

from sharejs.

The project pulls in the redis NPM package from its package.json.

Given that, the only way to check for redis availability is to try and connect to the database. If the connect fails, I don't think it makes sense to failover to a memory store - that would mean if you typed the wrong port in or something you could end up with no persistance (and somebody out there won't notice the warning messages in the log.)

I'm going to leave this as is.

from sharejs.