-
-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
npm install breaks on root #172
Comments
hmm, I think we should start by bumping the versions of the mattr lib, and seeing if that resolves the issue. |
I ran |
@OR13 I think it may be the specific version |
ohh....yeah... :( |
I just cloned fresh, and so far, I have not seen this. |
@decentralgabe you mind testing |
also node version, you need to be node14 for this mono repo, due to that library... its the main reason we still have node 14 recommendations at the top of this readme. |
I'm
|
See the warning at the top of our readme: https://github.com/transmute-industries/verifiable-data#node-verion That said, I think we should probably try to matrix out support for more than just node14... even if we have to nerf some of the wasm based tests related to the mattr lib |
same error with this command |
ah I'm on node 17.4 and npm 8.3.1, so I'll need to downgrade |
Hi @OR13, related to this issue, do you plan to update mattrglobal/node-bbs-signatures to a more recent version? |
@yhuard yeah.... we plan to update that... there is some breaking WASM issue between versions... its in our backlog... if you are brave, I welcome a PR. |
Hi @OR13 continuing on the discussion from last week I opened #204 updating the mentioned package. The bump would be ok for node 14
Can confirm that node 16 is not ok though
Although the upgrade is ok for 14, the wasm error for 16 is still present as mentioned. |
I suspect this is caused by "bad data"... and that the test data will need to be updated for the new library version. |
is this resolved?. I am currently block by this |
steps to repro:
git clone [email protected]:transmute-industries/verifiable-data.git
cd verifiable-data
npm i
this error shows up. have tried 3x, same stuff.
Trying to resolve https://github.com/mattrglobal/node-bbs-signatures/releases/download/0.11.0/node-v102-darwin-arm64.tar.gz directly also returns a 404
The text was updated successfully, but these errors were encountered: