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

DVN script added to LZ deployment #14

Merged
merged 71 commits into from
Sep 8, 2024
Merged

DVN script added to LZ deployment #14

merged 71 commits into from
Sep 8, 2024

Conversation

CarsonCase
Copy link
Collaborator

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

@CarsonCase
Copy link
Collaborator Author

CarsonCase commented Aug 19, 2024

ToDo:

  • Configure optional DVNs and Threshold
  • Add prompt for use of default
  • update the deployment script to set delegate properly
  • sort the dvns
  • revert if libraries not set
  • set receive and send libraries

@CarsonCase
Copy link
Collaborator Author

image

^ Example usage of the script

lzConfigCheck.cjs Outdated Show resolved Hide resolved
@CarsonCase CarsonCase changed the title DVM script added to LZ deployment DVN script added to LZ deployment Aug 22, 2024
@CarsonCase
Copy link
Collaborator Author

In Makefile have a big comment explaining deploy flow

@junkim012 junkim012 merged commit 12188d9 into master Sep 8, 2024
5 of 6 checks passed
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.

3 participants