-
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
New Arbitrum Token Bridge #1
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: sunbreak1211 <129470872+sunbreak1211@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for sending to an audit.
We should follow up on the router exclusivity issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good for audit
* Add L2-side sanity checks * Move loop to registerTokens * Sanity check L2 spell itself --------- Co-authored-by: telome <>
Add recommendations from Cantina
Remove redundant var in init script
Add Chainsecurity audit report
* Split L1 and L2 deploy scripts * Use single Deploy script with multiple deployer keys * Remove outdated files * Remove DEPLOYER env vars --------- Co-authored-by: telome <>
Co-authored-by: telome <>
telome seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
* Implement fileable escrow * Remove unused function from interface
Co-authored-by: telome <>
* Make gateways upgradable * Align version --------- Co-authored-by: telome <>
Co-authored-by: telome <>
* Add Certora specs * Improve PATH compatibility
No description provided.