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

Update bn.js to latest v4 version #2

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

Conversation

adrianmcli
Copy link

This updates bn.js to its latest v4 version (i.e. 4.11.8).

Motivated by the fact that Web3.js (one of the largest users of this library) is also using this latest version. Note that I've kept the version pinned and also updated the build artifacts.

v5 of bn.js is already released, so it's unlikely that there will be much change for v4 going forward.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 813ff12 on adrianmcli:update/bn.js into c3ff991 on SilentCicero:master.

@SilentCicero
Copy link
Owner

SilentCicero commented Nov 7, 2019 via email

@adrianmcli
Copy link
Author

@SilentCicero thx mang

@sakulstra
Copy link

@SilentCicero is there anything blocking a merge here?
Also would it make sense to losen the version to any v4 or v5?

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.

4 participants