-
Notifications
You must be signed in to change notification settings - Fork 834
EIP 7702 authority list signing utilities #3933
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
3f8b96f
to
7aa983f
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
I think we can delete example this https://github.com/ethereumjs/ethereumjs-monorepo/blob/tx-authlist-signing/packages/tx/examples/EOACodeTx.ts. Hmm no nvm it's referenced in the README/Changelog |
I ran the pectra tests locally, they pass. Open for review 👍 😄 |
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.
This is really marvellous, especially this setEOATx.ts
example! 🤩
Feel free to merge, needed to re-trigger a failing devp2p test.
Closes #3929
WIP
TODO:
vm/runTx
authorizationListJSONToBytes
to no work onauthorizationListItem[]
but instead work onauthorizationListItem
, because now there is this awkwardauthorizationListJSONToBytes([item])[0]
to convert an element. Also for Bytes -> JSON