Skip to content
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

deps: switch from stablelib to noble #185

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

paulmillr
Copy link

@paulmillr paulmillr commented Aug 4, 2024

The build was failing because of some low-level issues in Terser. I've updated devdeps to fix it:

@walletconnect/randombytes:build: ERROR in index.min.js from Terser
@walletconnect/randombytes:build: Error: error:0308010C:digital envelope routines::unsupported

Now tests in crypto/crypto are failing because of jsdom:

Exception during run: TypeError: Cannot set property crypto of #<Object> which has only a getter

Need to figure out why. My initial guess was that jsdom-globals is too old and doesn't support modern syntax. I've tried replacing it with more modern globals-jsdom but the package had a different kind of error

@paulmillr
Copy link
Author

@bkrem @glitch-txs @ganchoradkov could you approve CI run?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants