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.

Sign up for GitHub

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

Jump to bottom

refactor: simplify BOLDUpgradeAction #266

Merged
merged 38 commits into from
Nov 21, 2024
Merged

refactor: simplify BOLDUpgradeAction #266

merged 38 commits into from
Nov 21, 2024

Conversation

gzeoneth
Copy link
Member

@gzeoneth gzeoneth commented Nov 19, 2024 •
edited
Loading

This PR

  • Remove redundant ROLLUP_READER
  • Get rid of old rollup upgrade in BOLDUpgradeAction

@gzeoneth
chore: use 2.1.0 for OldRollup
62d2fbf
@gzeoneth
fix: wait for deployment
a6a03f5
@gzeoneth
feat: allow excessStakeReceiver to be eoa
fdc57f5
@gzeoneth
wip: print action calldata
60392fb
@gzeoneth
wip: show executor perform call data
d37a3b8
@gzeoneth
fix: testnode master rollup addr
5250976
@gzeoneth
wip: add local testnode executor
d13ff1b
@gzeoneth
wip: testnode validator
054c29b
@gzeoneth
wip: testnode validator wallet
74b1a06
@gzeoneth
fix: small range lookup
980d2f8
@gzeoneth
chore: sample env for testnode
d766a61
@gzeoneth
feat: bold upgrade execute and improved logging
4806092
@gzeoneth
fix: use nextInboxPosition at upgrade to verify
6948823
@gzeoneth
Revert "fix: use nextInboxPosition at upgrade to verify"
8d9e42e 
This reverts commit 6948823.
@gzeoneth
fix: getlog range
716f3ef
@gzeoneth
fix: nextInboxPosition
d7cfe47
@gzeoneth
chore: print bold addresses
46f9e27
@gzeoneth
chore: log more stuffs
a6e2e59
@gzeoneth
chore: use 1000 range when searching log
4995461
@gzeoneth
fix: parse event to get nextInboxPosition and more logging
ef08bac
@gzeoneth
feat: allow ROLLUP_ADDRESS override in bold upgrade
7d02843
@gzeoneth
refactor: remove rollup reader
55ddba3
@cla-bot cla-bot bot added the s label Nov 19, 2024
@gzeoneth gzeoneth changed the title refactor: remove rollup reader refactor: simplify BOLDUpgradeAction Nov 20, 2024
yahgwai
yahgwai approved these changes Nov 20, 2024
View reviewed changes
scripts/executeBoldUpgrade.ts Show resolved Hide resolved
@gzeoneth gzeoneth mentioned this pull request Nov 21, 2024
Merged
Base automatically changed from bold-merge-script to bold-merge November 21, 2024 09:54
@gzeoneth gzeoneth marked this pull request as ready for review November 21, 2024 09:59
@gzeoneth gzeoneth merged commit 59d3cbd into bold-merge Nov 21, 2024
9 checks passed
@gzeoneth gzeoneth deleted the bold-merge-action branch November 21, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yahgwai yahgwai yahgwai approved these changes

Assignees
No one assigned
Labels
s
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gzeoneth @yahgwai