Skip to content
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

Deployment script: fix the addresses output #405

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

bpierre
Copy link
Contributor

@bpierre bpierre commented Sep 3, 2024

(reapply & adapt changes removed from this PR: d4b71ef)

The script now separates the protocol addresses from the each group of collateral addresses.

This commit also updates the deployment-artifacts-to-app-env script to generate env vars separated in a similar way.

Example output:

$ ./deploy local --open-demo-troves

Deploying Liquity contracts with the following settings:

  CHAIN_ID:           31337
  DEPLOYER:           0xac0974bec39a17e36ba4a6b4d238ff944bacb478cbed5efcae784d7bf4f2ff80
  LEDGER_PATH:        undefined
  ETHERSCAN_API_KEY:  undefined
  OPEN_DEMO_TROVES:   yes
  RPC_URL:            http://localhost:8545
  VERIFY:             no
  VERIFIER:           etherscan
  VERIFIER_URL:       undefined

Protocol contracts:

  WETHTester          0x618ab3160c5bbbc5bfa034aca8e37df3eea0316d
  BoldToken           0x2710f73233d0077023703bbb5835abb3d696e459
  CollateralRegistry  0xe082678ecf749982e33ea6839852a8cd989aede2
  HintHelpers         0x984f797d26d3da2e9b9f8ae4eefeacc60fcaa90c
  MultiTroveGetter    0x1ec5df6299467fb19b4e425eb47df46c8f245078

Collateral 1 contracts:

  activePool          0xFB8d5B2c03cc732657fbA00624e8b442E382d295
  borrowerOperations  0xE81687523a0FbfAD4708Aa77d17c603814AB0DC6
  sortedTroves        0x3A8e768F2B74Fd5589565c45A0ffAEa476a4956d
  stabilityPool       0xCE2aC8d208D8f0f2b68B978eb6908Ea63f4bd775
  token               0x618aB3160c5bbBc5bFa034ACa8e37dF3Eea0316D
  troveManager        0x182689ba50e73Aa24fa06d5EB796264adeBC4664

Collateral 2 contracts:

  activePool          0x934B7e2AA0485D38FDD05172Fd7f8c4c77731342
  borrowerOperations  0x2eD632b28874Bc89c7CEf510c325c1dFDBE5bb6d
  sortedTroves        0x5d170C4Eb0d8fDd8e03e4f2AdF8815b92B7BcAcb
  stabilityPool       0x7cd1C56dF3CaaCcccF33c35B20e46D283fd108ee
  token               0x82C9d5501a2C46125A82cc82D0881FA27751EA3E
  troveManager        0x83e1C2C7f647De452A7E5f2A37F1fAd882Ea97EE

Deployment complete.

The script now separates the protocol addresses from the each group of
collateral addresses.

This commit also updates the deployment-artifacts-to-app-env script to
generate env vars separated in a similar way.
Copy link
Collaborator

@danielattilasimon danielattilasimon 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! 👍 👍

@danielattilasimon
Copy link
Collaborator

Going to merge this as it'll come in handy for the Sepolia deployment tonight.

@danielattilasimon danielattilasimon merged commit c845858 into main Sep 4, 2024
6 of 7 checks passed
@bpierre bpierre deleted the deploy-contracts-output branch September 24, 2024 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants