node-memwatch icon indicating copy to clipboard operation
node-memwatch copied to clipboard

Please mark as deprecated

Open JaredCE opened this issue 5 years ago • 4 comments

Code hasn't been updated since 2017 and doesn't work on latest versions of Node. Please mark this project as deprecated.

JaredCE avatar May 07 '20 09:05 JaredCE

Agreed. use this instead: https://github.com/airbnb/node-memwatch

Genysys avatar Jun 17 '20 10:06 Genysys

@Genysys looks like there is no leak monitor event

cool-firer avatar Mar 12 '21 06:03 cool-firer

@Genysys looks like there is no leak monitor event

Yes, airbnb's fork removed that part of the code. There explanation was given here.

And here is the code that was removed that was doing the leak detection: https://github.com/airbnb/node-memwatch/pull/1/files#diff-16ab447da591c718a41d43cb70d0f471069f1db23629b3751d8d7af891b20c13L108

It should be possible to re-implement that behavior in userland JS code. If someone does this transformation (it doesn't look too hard), please post the JS code for it here. :)

Venryx avatar Jan 12 '22 01:01 Venryx

~Hmm, the airbnb fork for me seems to be failing to build on Node v14.17.1 -- at least on Windows.~

Have others gotten it to work on Node v14+, on either Linux or Windows?

EDIT: While it failed to install on Windows, that's fine since I don't need the module on my host computer, only in the Docker container (which runs Linux). When installing+running in Linux, the module worked fine, with FROM node:14-alpine as the base image.

Venryx avatar Jan 12 '22 07:01 Venryx