per instance Scripty cache

[behrad]

using a global cache https://github.com/TheDeveloper/scripty/blob/master/lib/Scripty.js#L16 prevents from closing the redis client and create a new Scripty with a new redis client object.

[TheDeveloper]

is your new redis client connected to a different redis server?

[behrad]

no the same server, but different object instance, I'm closing old instance and creating new one :)
this is happening when gracefully shutdown kue: Automattic/kue#578

[behrad]

look at this example code: TheDeveloper/warlock#11 (comment)

[behrad]

generally caching a user/client provided instance object (the redis client) is a bad idea along side Script object.

[TheDeveloper]

oh yes I see, the redis client object is being cached along with the script. will merge your PR.

[behrad]

thank you @TheDeveloper

[behrad]

when can I switch to your npm?

[TheDeveloper]

just published v0.0.5 :)

warlock will pick up the latest version

[behrad]

👍