-
Notifications
You must be signed in to change notification settings - Fork 0
feat_: introduce ethclient #6
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
base: master
Are you sure you want to change the base?
Conversation
46062d7
to
85ceaad
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6 +/- ##
===========================================
- Coverage 87.28% 51.20% -36.09%
===========================================
Files 5 16 +11
Lines 236 1125 +889
===========================================
+ Hits 206 576 +370
- Misses 26 527 +501
- Partials 4 22 +18
🚀 New features to boost your workflow:
|
e83cde8
to
a7ee8ac
Compare
} | ||
|
||
// Real block JSON data | ||
blockJSON := `{ |
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.
What do you think about putting this in a separate json file and loading it dynamically?
"github.com/ethereum/go-ethereum/common/hexutil" | ||
"github.com/ethereum/go-ethereum/rpc" | ||
) | ||
|
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.
It would be nice to have explanatory comments on these methods as we have in all other files
Looks great! 🎉 |
Introduce types and a convenience client to interact with the standard Ethereum JSON-RPC interface.
go-ethereum's types contain some additional logic (calculate hash based on the contents of a Block/Transaction, for example) which makes them inconvenient to use across different L1s and L2s. This implementation consumes all (and only) information provided in the JSON-RPC responses with no extra assumptions.