Skip to content

feat: svm deploy instructions #1006

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

Merged
merged 5 commits into from
May 26, 2025
Merged

feat: svm deploy instructions #1006

merged 5 commits into from
May 26, 2025

Conversation

Reinis-FRP
Copy link
Contributor

@Reinis-FRP Reinis-FRP commented May 19, 2025

This updates scripts and readme for SVM deploy & verification with multisig

@Reinis-FRP Reinis-FRP changed the title fix: remove keypairs only in ci feat: svm deploy scripts May 19, 2025
Signed-off-by: Reinis Martinsons <[email protected]>
@Reinis-FRP Reinis-FRP force-pushed the reinis-frp/svm-deploy branch from 413e7af to 3abb91a Compare May 21, 2025 10:53
@Reinis-FRP Reinis-FRP changed the title feat: svm deploy scripts feat: svm deploy instructions May 21, 2025
Signed-off-by: Reinis Martinsons <[email protected]>
Signed-off-by: Reinis Martinsons <[email protected]>
Signed-off-by: Reinis Martinsons <[email protected]>
Signed-off-by: Reinis Martinsons <[email protected]>
@Reinis-FRP Reinis-FRP marked this pull request as ready for review May 23, 2025 14:49
@@ -29,17 +30,18 @@ const argv = yargs(hideBin(process.argv))
.option("fillDeadlineBuffer", {
type: "number",
demandOption: false,
default: 3600 * 4,
default: 3600 * 6,
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this the value we use in EVM?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, we have 21600 in other spoke pools

Copy link
Contributor

@md0x md0x left a comment

Choose a reason for hiding this comment

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

Looks good!

@Reinis-FRP Reinis-FRP merged commit 8355870 into master May 26, 2025
17 checks passed
@Reinis-FRP Reinis-FRP deleted the reinis-frp/svm-deploy branch May 26, 2025 13:32
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