Skip to content
This repository was archived by the owner on Apr 12, 2021. It is now read-only.

deploy-script: add heights#51

Closed
tynes wants to merge 2 commits intomasterfrom
deploy-heights
Closed

deploy-script: add heights#51
tynes wants to merge 2 commits intomasterfrom
deploy-heights

Conversation

@tynes
Copy link
Collaborator

@tynes tynes commented Nov 5, 2020

Description

Adds the deploy height to the output of the deploy script. This is useful for booting up geth because the CTC deploy height is a config param. This is used as the height of the L1 chain to start syncing from. To get this, I needed to get the receipt for each contract being deployed. This required me to update the interfaces to some functions.

Contributing Agreement

@tynes tynes requested a review from karlfloersch November 5, 2020 20:17
Copy link
Contributor

@K-Ho K-Ho left a comment

Choose a reason for hiding this comment

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

Lgtm!

@tynes
Copy link
Collaborator Author

tynes commented Mar 3, 2021

Too old at this point

@tynes tynes closed this Mar 3, 2021
@smartcontracts smartcontracts deleted the deploy-heights branch March 25, 2021 17:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants