-
Notifications
You must be signed in to change notification settings - Fork 25
Lodestar should pass fee market test #61
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
Changes from all commits
f61d9ba
9b27230
5fb61e5
e5e0351
48fe02f
042c4eb
5fe616d
ebccdf8
f8f5d42
2b5e083
6b04bac
75c78da
ea6482c
c7b628a
591a01b
04aa67d
b4f19b8
e8bf85c
b3c004d
05d7355
7da4527
109b469
d39f2e9
ff52ace
9bf202f
0b427c7
4eaf186
ad8a2ee
c9a1883
8fe2d40
002b39a
9573bb2
44b2b3f
4a6ee67
d7fa20e
a23b3f6
ee5b6c7
4ed4ce9
1f7cb34
91ce87e
b1affe4
52f0c08
12154e7
97285e2
d6fab97
fa5537e
aab8a1c
334ea3e
a6e3b3d
8a6ceb0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,4 @@ | ||
| .git | ||
| .github | ||
| node_modules | ||
| **/node_modules | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -51,7 +51,9 @@ func WaitForNextSlots(ctx context.Context, client *beacon.Client, slots consensu | |
| } | ||
|
|
||
| type Body struct { | ||
| BlobKzgs []string `json:"blob_kzgs"` | ||
| // Does Prysm call this "blob_kzgs"? That's wrong. | ||
| // BlobKzgs []string `json:"blob_kzgs"` | ||
| BlobKzgs []string `json:"blob_kzg_commitments"` | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Prysm should be updated?
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yup good catch, let me fix.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| } | ||
|
|
||
| type Message struct { | ||
|
|
||
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 appears to be necessary to avoid copying the compiled kzg.node, which won't work if you have an OSX host, and the docker is Linux.