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