-
Notifications
You must be signed in to change notification settings - Fork 376
Truffle v5 support #199
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
Truffle v5 support #199
Conversation
Truffle (at least version 5.0.x) use `BN` internally, not BigNumber.
Replace BigNumber with BN.js
…get-truffle-v5, because @ character is not allowed in package names.
…get-truffle-v5, because @ character is not allowed in package names.
Add truffle v5 target, switch from BigNumber to BN
Hi @krzkaczor, what is left to be done here? I tried locally the branch this PR is based on and it worked great :) |
@PetarKirov thanks for testing. It should work. The todo is in the PR description, mostly its "infra" stuff as I ran into problems with releasing new packages on NPM. Sorry for the drag, it should be released in the next upcoming days. |
No worries, thanks for working on this nice project! |
It's released as |
I installed @typechain/truffle-v5, but I'm not sure how to update the core package to make use of the new target... Is this available on npm already? |
@TripleSpeeder sorry, this branch is a hot mess right now. Some pointers are here: dethcrypto/truffle-typechain-example#7 |
This is being merged into #219 It will done really soon! (~1 week), I need to write some tests and add support for overloaded events/methods. |
typechain-target-truffle-v4
@typechain/truffle-v5