mapbox / node-pre-gyp

Node.js tool for easy binary deployment of C++ addons
BSD 3-Clause "New" or "Revised" License
1.11k stars 260 forks source link

upgrade nopt and add `engines` field #748

Closed benmccann closed 3 months ago

benmccann commented 3 months ago

requires a higher version of node. let's upgrade it manually so that we can specify the engines requirement https://github.com/npm/nopt/blob/main/CHANGELOG.md#700-2022-11-02

edit: actually, it seems some of the other dependencies such as tar require Node 18, so we need to specify higher versions that what nopt requires