robertklep / node-metrohash

Node.js bindings for MetroHash
MIT License
26 stars 8 forks source link

Updated to latest node-gyp and mocha. Fixes Python v3 compat. #18

Closed swernerx closed 3 years ago

swernerx commented 4 years ago

We have found out the metrohash stops us from migrating our repository which uses a few other node-gyp enabled packages to Python v3. The previously used version of node-gyp inside this project only supported Python 2.7.

Looks like all tests are still green.

s5bug commented 3 years ago

For anyone else who is running into this issue because they're using https://github.com/sebastian-software/rollup-plugin-rebase (or any other project that depends on metrohash), adding:

  "resolutions": {
    "metrohash": "git://github.com/swernerx/node-metrohash.git#master"
  }

to your package.json, then cleaning node_modules/metrohash and node_modules/node-gyp if they exist, will get the project to build correctly.

robertklep commented 3 years ago

Just published metrohash@2.8.0 which uses the latest of everything.