Skip to content
This repository has been archived by the owner on May 28, 2024. It is now read-only.

Get rid of node-hashes #30

Open
akshat1 opened this issue Feb 3, 2017 · 0 comments
Open

Get rid of node-hashes #30

akshat1 opened this issue Feb 3, 2017 · 0 comments
Assignees

Comments

@akshat1
Copy link
Owner

akshat1 commented Feb 3, 2017

node-hashes uses node-gyp and is often problematic in CI environments. It won't work on Travis which is stuck on node 6, and it won't work on my own jenkins unless build-essential is installed; and then it will throw warnings about deprecated C++.

Considering we only use JSHashes, it might be a good idea to simply extract this function in a pure JS module and use that.

@akshat1 akshat1 self-assigned this Feb 3, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant