-
Notifications
You must be signed in to change notification settings - Fork 7
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
DVN script added to LZ deployment #14
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ToDo:
|
jpick713
reviewed
Aug 19, 2024
script/deploy/single/05b_DeployMultiChainLayerZeroTellerWithMultiAssetSupport.s.sol
Outdated
Show resolved
Hide resolved
…to deployment specific config
jpick713
approved these changes
Aug 22, 2024
CarsonCase
changed the title
DVM script added to LZ deployment
DVN script added to LZ deployment
Aug 22, 2024
merging in bug fix from other class
In Makefile have a big comment explaining deploy flow |
CarsonCase
commented
Sep 6, 2024
script/deploy/single/05b_DeployMultiChainLayerZeroTellerWithMultiAssetSupport.s.sol
Outdated
Show resolved
Hide resolved
Carson/decimal testing
Integration Test
Carson/form deployment
chore: delete accidentally included vscode file
…oring-vault into carson/dvm-script pull
junkim012
approved these changes
Sep 8, 2024
jpick713
approved these changes
Sep 8, 2024
jpick713
approved these changes
Sep 8, 2024
junkim012
approved these changes
Sep 8, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
It checks if a default config exists, if not, it then deploys based off a DVN provided in the config as well as a confirmation number provided there.
This config does not allow for multiple DVNs or optional ones. But our current plan seems to be to just use LZ alone if this happens and we can always update this again via multisig