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

Fix truffle-contacts dependencies #1646

Merged

Conversation

maksim-tolo
Copy link
Contributor

  • Add missed ethers and truffle-compile dependencies
  • Remove unused ethereumjs-util, ethjs-abi, lodash and require-nocache dependencies
  • Move uglify-es to devDependencies
  • Replace web3.utils with web3-utils
  • Replace ethers.utils.bigNumberify with require('ethers/utils/bignumber').bigNumberify

Production bundle size has been reduced by more than 264 KB (≈12%).

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.006%) to 69.894% when pulling 60dee61 on maksim-tolo:truffle-contract-dependencies-fix into d15834b on trufflesuite:develop.

Copy link
Contributor

@gnidan gnidan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! Nice job digging and sorting all this out!

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.

3 participants