-
Notifications
You must be signed in to change notification settings - Fork 2
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
Deploy to Goerli and Optimism Goerli #80
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
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, a couple small things that shouldnt block merge
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
f1ee4c4
to
4461e27
Compare
2732b80
to
1b671e8
Compare
36fdc6c
to
e4df2f6
Compare
1b671e8
to
bf6ccf8
Compare
Coverage after merging feature/support-other-testnets into feature/find-test-gaps will be
Coverage Report
|
* Deploy to Goerli and Optimism Goerli * Add missing sample environment variables
Description
Closes #79