Skip to content

Conversation

@leolara
Copy link
Member

@leolara leolara commented Nov 25, 2025

No description provided.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please run make lint to format the file.


#### Expectation

Bid is accepted, latest_execution_payload_bid is updated in state, and no pending payment is created since value is zero.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are many instances of inline code which are not shown as code. Eg latest_execution_payload_bid.


#### Expectation

Bid is accepted, latest_execution_payload_bid is updated, builder balance remains unchanged, and a pending payment is created in the builder_pending_payments queue with weight = 0 and withdrawable_epoch = FAR_FUTURE_EPOCH.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer we don't do the "variable = value" thing & instead write it as a proper sentence. Mostly for consistency; see the scenario section immediately above which does this.

@jtraglia jtraglia changed the title Add checklist for EIP7732 Add test checklist for eip7732 Nov 26, 2025
@leolara
Copy link
Member Author

leolara commented Nov 28, 2025

This PR is still in draft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing CI, actions, tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants