-
Notifications
You must be signed in to change notification settings - Fork 163
CLI support for Arbitrum bridge + L2 rewards distribution #556
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
Conversation
9447437 to
da9199b
Compare
1deec1a to
94b5327
Compare
cf6d53f to
e77fe37
Compare
cac6195 to
822e1da
Compare
e77fe37 to
12bac31
Compare
f6a86e4 to
70218ec
Compare
12bac31 to
053ef34
Compare
9b7f401 to
a5342c1
Compare
053ef34 to
f5a5f24
Compare
a5342c1 to
62e9270
Compare
f5a5f24 to
87f4bec
Compare
e60da74 to
f80cad3
Compare
87f4bec to
edbc3e0
Compare
|
Marking this as ready for review, but we should merge #552 before this one. |
f80cad3 to
0495b9c
Compare
795effd to
6f70096
Compare
Codecov Report
@@ Coverage Diff @@
## pcv/arb-bridge #556 +/- ##
==================================================
+ Coverage 91.32% 91.92% +0.60%
==================================================
Files 40 44 +4
Lines 1959 2080 +121
Branches 338 356 +18
==================================================
+ Hits 1789 1912 +123
+ Misses 170 168 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
We also add Goerli and Arbitrum Nitro Devnet configurations (though the CLI commands don't support Nitro yet).
…-merge-552-fixes This includes all the audit fixes on PR #552.
Socket Security ReportDependency issues detected. If you merge this pull request, you will not be alerted to the instances of these issues again. 📜 New install scripts detectedA dependency change in this PR is introducing new install scripts to your install step.
🫣 Native codeContains native code which could be a vector to obscure malicious code, and generally decrease the likelihood of reproducible or reliable installs.
Socket.dev scan summary
Powered by socket.dev |
Following from #552, this PR adds support for some commands to configure and use the Arbitrum GRT bridge. We include commands to configure the bridge, and commands to send tokens from L1 to L2 and back from L2 to L1.
This PR temporarily targets the
pcv/arb-bridgebranch, until #552 is merged.Edit: merged #571 onto this, so this now also includes the changes to support the new rewards distribution to L2.