Add support for Node 17 and 18. Drop support for Node 10, 12 and 15. about pprof-nodejs

[ericwooley]

Should this not have 20 as well now?

[franleplant]

would love to see support for node@20 20

[nolanmar511]

Might be possible to switch to using Node.js's inspector & drop the native portion of this library once LTS for Node 12 has ended.

[pietrodn]

Sorry, I realized that somehow the binary I had in node_modules was built against the wrong Node runtime. A reinstall did the trick.

[amchiclet]

Revisiting this issue now. Updating the title so it matches the current releases of node.

The plan is to update the tests for node 17 and 18 and drop tests for node 10 and 15.

Would also like to note that since we use source-map, the tests for node 18 will fail until the upstream issue is officially fixed.

A workaround is to use the 0.8.0-beta.0 version of source-map.

[nolanmar511]

node-pre-gyp does not currently have maintainers (mapbox/node-pre-gyp#657) and does not support building node 18 binaries on a machine using another version of node (mapbox/node-pre-gyp#647).

Given this, the approach for generating pre-built binaries will need to change to build binaries for Node 18.

[aabmass]

Should also update the docs here https://cloud.google.com/profiler/docs/profiling-nodejs

[pietrodn]

Would it be possible to cut a release to have Node 18 support on the latest stable version? Thanks.

[aabmass]

This library works fine on node 18 already. The only issue is we can't supply pre-built binaries. Are you seeing a specific issue @pietrodn ?

[aabmass]

It should work fine in node 20 already. Please see #273 (comment), for more info.