-
Notifications
You must be signed in to change notification settings - Fork 3
Updated to work under Node.js v12 #1
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
base: master
Are you sure you want to change the base?
Conversation
Thanks @vlasky, I want to rather move away from NAN entirely in favor of N-API. |
I'm on the waitlist 🖐 because of this version incompatible with node@12. |
Can we merge this fix then move away from NAN in a separate PR? |
Updated nan version to 2.14.1
There hasn't been any activity on this project for a while, so to help other people who want this working, I have released my fork as an NPM named @vlasky/quoted-printable. |
awesome thanks man, very helpful |
- Update NAN from 2.14.0 to 2.22.2 (officially supports Node.js 8-22) - All tests pass on Node.js 24.2.0 - Update @ronomon/queue from 3.0.0 to 3.0.1 - Add engines field requiring Node.js >=8.0.0 - Update description and README to reflect Node.js 8+ support
Updated to use the latest version of nan (2.14.0). Tweaks were made to the semantics of Uint32Value() to fix compilation errors.