-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update basic-ops docs #159
Conversation
|
||
```javascript | ||
import { iOVM_L1BlockNumber } from "@eth-optimism/contracts/L2/predeploys/iOVM_L1BlockNumber.sol"; | ||
import { Lib_PredeployAddresses } from "@eth-optimism/contracts/libraries/constants/Lib_PredeployAddresses.sol"; | ||
import { L1Block } from "@eth-optimism/contracts-bedrock/L2/L1Block.sol"; |
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.
@souradeep-das does this look right? I briefly tried to get this to build on my local machine, but ran into errors that I think are largely related to my incompetence.
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #159 +/- ##
===========================================
+ Coverage 18.95% 28.21% +9.26%
===========================================
Files 158 168 +10
Lines 6226 7294 +1068
Branches 1311 1311
===========================================
+ Hits 1180 2058 +878
- Misses 4971 5130 +159
- Partials 75 106 +31
Flags with carried forward coverage won't be shown. Click here to find out more. |
Co-authored-by: Souradeep Das <[email protected]>
96a9246
to
948ff11
Compare
Updates the basic-ops page to reflect Anchorage.