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

Remove native dependency #108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nebojsa94
Copy link

This PR removes bip39 package and instead uses js-ethereum-cryptography

@alcuadrado
Copy link

While we understand that this version of the package has been deprecated for some time, it's still frequently used. Can you please make an exception here and release a minor/patch version, @gnidan? 🙏

The problem is that if a single (popular) dependency installs a native module that requires recompilation, users without a proper setup (i.e. most new users) will run into problems.

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants