-
Notifications
You must be signed in to change notification settings - Fork 129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
support nodejs 6.x (update nan, use node-gyp) #93
base: master
Are you sure you want to change the base?
Conversation
dankang
commented
Jun 22, 2016
- update nan version (2.1.0 => 2.3.5)
- use node-gyp instead of pangyp(deprecated)
- tested on nodejs 4.4.5 & 6.2.2
* update nan version (2.1.0 => 2.3.5) * use node-gyp instead of pangyp(deprecated) * tested on nodejs 4.4.5 & 6.2.2
@kuno |
Fixed node 6 deprecation warnings
@kuno is there any chance to get this merged? There are multiple security issues in the |
+1 |
node v6 has moved into active LTS mode and a lot of people can't upgrade their projects (myself included) because we have a dependency on this package. This PR really needs to be rolled in at least to make it possible to upgrade. |
@kuno - what is the status on this ?. We have an urgent need to have this mergerd due to nsp checks
|
I needed this so I created |