-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Detailed description
Currently, this project is using Truffle v5.
Line 49 in 2abee70
"truffle": "5.4.1",
But Typechain is targeted as @typechain/truffle-v4.
Line 7 in 2abee70
"generate": "yarn compile && typechain --target=truffle-v4 './build/**/*.json'",
Line 28 in 2abee70
"@typechain/truffle-v4": "4.1.0",
We need to use @typechain/truffle-v5.
Context
Changing the Typechain version should result in many type errors, but as long as the existing test cases pass the CI, the existing codes are certainly executable code.
So, we will need to change function callings or type inferences to accommodate the new types, but basically the assertions and behavior itself should not need to change.
Possible implementation
No response
Additional information
As I tried it locally... it seems like the following major changes are needed:
- LockupInstance type is no longer exports
depositToProperty
, it exportsmethods['depositToProperty(address,uint256)']
andmethods['depositToProperty(address,uint256,bytes32)']
instead of it. - Truffle is no longer uses BigNumber.js, it uses bn.js instead of it. But we only use the interfaces common to both and test cases have no problem.
Code of Conduct
- I agree to follow this project's Code of Conduct
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request