Skip to content
This repository has been archived by the owner on Aug 6, 2021. It is now read-only.

APm-45 rewrite deploy #51

Merged
merged 48 commits into from
Aug 31, 2020
Merged

APm-45 rewrite deploy #51

merged 48 commits into from
Aug 31, 2020

Conversation

vkonst
Copy link
Contributor

@vkonst vkonst commented Aug 31, 2020

Task #APm-45 done

Rewrite deploy-contracts.js to use ethers.js / web3.js only
We want to do this because truffle-contracts causes issues during deployment

Deployments scripts re-written

Based on buidler-deploy plugin, new deployment scripts:

  • avoid unnecessary re-deployments of smart-contracts
  • save extensive deployments details
  • failure-proof
  • idempotent (may be re-started/re-run)

'truffle' removed from dependencies

(except yet for actus-solidity)
truffle is no longer a dependency

Tests updated

(except yet for actus-solidity)

  • EVM snapshots used whenever possible
  • "pure" web3.js Contract abstraction/interface used

CI- and automation scripts updated

# Conflicts:
#	packages/ap-contracts/buidler.config.js
[skip ci]
(#APm-45)
# Conflicts:
#	packages/ap-contracts/deployments.json
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
[skip ci]
(#APm-45)
# Conflicts:
#	packages/ap-contracts/test/Core/ANN/ANNActor/TestANNEngine.js
#	packages/ap-contracts/test/Core/CEC/CECActor/TestEnhancements.js
#	packages/ap-contracts/test/Core/PAM/PAMActor/TestPAMEngine.js
#	packages/ap-contracts/test/helper/setupTestEnvironment.js
@vkonst vkonst changed the title A pm 45 rewrite deploy APm-45 rewrite deploy Aug 31, 2020
@vkonst vkonst requested a review from jo-es August 31, 2020 03:09
@jo-es jo-es merged commit b74c169 into dev Aug 31, 2020
@jo-es jo-es deleted the APm-45-rewrite-deploy branch September 29, 2020 09:17
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.

2 participants