-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
fix(devnet): Updated the L2 address to 9545 #5261
Conversation
|
✅ Deploy Preview for opstack-docs canceled.
|
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. Confirmed l2 geth port 8545 is exposed to the host as port 9545 so that's the port it should be accessed on.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #5261 +/- ##
===========================================
- Coverage 39.78% 35.90% -3.88%
===========================================
Files 381 226 -155
Lines 24198 19666 -4532
Branches 837 0 -837
===========================================
- Hits 9628 7062 -2566
+ Misses 13844 11916 -1928
+ Partials 726 688 -38
Flags with carried forward coverage won't be shown. Click here to find out more. |
This PR has been added to the merge queue, and will be merged soon. |
This PR is next in line to be merged, and will be merged as soon as checks pass. |
1 similar comment
This PR is next in line to be merged, and will be merged as soon as checks pass. |
That's the correct value, not 8546 (which is, IIRC, websocket for L1).