search

redis / hiredis-node

Node wrapper for hiredis
BSD 3-Clause "New" or "Revised" License
305 stars 84 forks source link

Update for NAN 2.0 #96

Closed kkoopa closed 9 years ago

kkoopa commented 9 years ago

Fixes #95.

badboy commented 9 years ago

someone took care of it \o/ Thanks, I will check that out next week.

bcoe commented 9 years ago

@badboy, @kkoopa excited to get this merged so that we can run it against the node-redis test-suite :thumbsup: awesome work getting this patched so fast.

badboy commented 9 years ago

I read it and all looks good so far. I'm just waiting for appveyor to not be broken anymore and then I will merge and release it.

bcoe commented 9 years ago

@badboy mind pushing a release to npm when you have a moment?

badboy commented 9 years ago

Done, thanks @kkoopa.

I will automate the process in the future, so releases will happen more easily and faster.